Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix license header in core files #2648

Open
wants to merge 1 commit into
base: release-1.2
Choose a base branch
from
Open

Fix license header in core files #2648

wants to merge 1 commit into from

Conversation

Yury-Fridlyand
Copy link
Collaborator

Minor code grooming

@Yury-Fridlyand
Fix license header in core files
da4b624 
Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
@Yury-Fridlyand Yury-Fridlyand added the Rust core redis-rs/glide-core matter label Nov 8, 2024
@Yury-Fridlyand Yury-Fridlyand requested a review from a team as a code owner November 8, 2024 03:02
@avifenesh
Copy link
Collaborator

Note that CI is red

@Yury-Fridlyand
Copy link
Collaborator Author

I know
I don't understand why fmt blames if file has an inner attribute right after /** comment. Maybe I have to change license header format to /* thoughout.
BTW I can add headers to redis-rs files - this PR is a right place. Can/should we do that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@avifenesh avifenesh avifenesh approved these changes

Assignees
No one assigned
Labels
Rust core redis-rs/glide-core matter
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Yury-Fridlyand @avifenesh