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.

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

Ignore COPYING and LICENSE as exact matches or with extensions #886

Merged
merged 1 commit into from
Apr 27, 2024

Conversation

carmenbianca
Copy link
Member

Related to #881 and fsfe/reuse-docs#139

@mxmehl
Copy link
Member

mxmehl commented Jan 9, 2024

Uh, this will break things. I see many LICENSE.md or LICENSE.txt files around.

I don't quite understand why this change is needed. Do you want to avoid that a file such as LICENSE-finder.py is ignored? In this case, we could find a more elaborate regex.

@carmenbianca carmenbianca changed the title Ignore COPYING and LICENSE as exact matches Ignore COPYING and LICENSE as exact matches or with extensions Jan 16, 2024
@carmenbianca
Copy link
Member Author

carmenbianca commented Jan 16, 2024

Uh, this will break things. I see many LICENSE.md or LICENSE.txt files around.

Good shout. I changed this so that COPYING and COPYING.any_ext_here are ignored (ditto for LICENSE), but COPYING_foobar is not ignored.

CHANGELOG.md Outdated
Comment on lines 70 to 73
- Previously, any file that begins with `COPYING` or `LICENSE` was ignored. This
has been changed. Now, files like `COPYING_README` are no longer ignored, but
`COPYING` and `COPYING.txt` are still ignored (in other words: exact matches,
or `COPYING` + a file extension). Idem ditto for `LICENSE`. (#886)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is now in the 3.0.0 change log section…

Copy link
Member

@mxmehl mxmehl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this issue :)

Signed-off-by: Carmen Bianca BAKKER <carmenbianca@fsfe.org>
@carmenbianca carmenbianca merged commit a55d195 into fsfe:main Apr 27, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants