-
-
Notifications
You must be signed in to change notification settings - Fork 596
Update licenses and add rules #4278
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.
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Reference: #4173 Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
These were added by @DennisClark Reference: #4173 Reference: #4030 Reference: #4169 Reference: #4198 Reference: #4197 Reference: #4251 Reference: #4036 Reference: #4152 Reference: #4208 Reference: #4252 Reference: #4231 Reference: #4218 Reference: #4172 Reference: #4247 Reference: #4202 Reference: #4239 Reference: #2779 Reference: #4250 Reference: #4211 Reference: #4226 Reference: #4248 Reference: #4232 Reference: #4249 Reference: #4205 Reference: #4200 Reference: #4193 Reported-by: @armijnhemel Reported-by: @pombredanne Reported-by: @ariel11 Reported-by: @carolinamarc Reported-by: @LeChasseur Reported-by: @tsteenbe Reported-by: @stefan6419846 Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
These are added from real worlds usages on github for the added licenses. Reference: #4173 Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
pombredanne
approved these changes
Apr 28, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking fine. Good to merge once the tests pass!
0b4406f
to
c7ce521
Compare
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
c7ce521
to
849f975
Compare
pombredanne
approved these changes
May 11, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! merging.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Thanks to @armijnhemel, @pombredanne, @ariel11, @carolinamarc, @LeChasseur, @tsteenbe, @stefan6419846 for reporting these and @DennisClark for adding these. 🙇
Reference: #4173
Reference: #4030
Reference: #4169
Reference: #4198
Reference: #4197
Reference: #4251
Reference: #4036
Reference: #4152
Reference: #4208
Reference: #4252
Reference: #4231
Reference: #4218
Reference: #4172
Reference: #4247
Reference: #4202
Reference: #4239
Reference: #2779
Reference: #4250
Reference: #4211
Reference: #4226
Reference: #4248
Reference: #4232
Reference: #4249
Reference: #4205
Reference: #4200
Reference: #4193
Reference: #4253
Reference: #4254
Reference: #4173
Reference: #4254
Reference: #4257
Reference: #4255
Tasks
Run tests locally to check for errors.