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

License filename matching: check for word boundaries around the word 'license' or 'licence' #127

Closed
wants to merge 6 commits into from
Closed

License filename matching: check for word boundaries around the word 'license' or 'licence' #127

wants to merge 6 commits into from

Conversation

jayaddison
Copy link

Resolves #126.

src/LicenseTextReader.ts Outdated Show resolved Hide resolved
…ce' when looking for potential license files in project directories"

This reverts commit b673731.
… looking for potential license files in project directories

This is an alternative implementation of commit b673731.
…ry-check

Interleaves implementation approaches from commit b673731 and commit 1ea317b.
@jayaddison
Copy link
Author

Ready for re-review when you have time, @xz64

@jayaddison jayaddison requested a review from xz64 July 27, 2023 15:40
@jayaddison
Copy link
Author

I think that it would have been more appropriate for me to open this pull request from a fork of this repository that belongs to the openculinary organization on GitHub (since it affects code developed in my capacity as an employee of it), so I'm going to close this and re-open it from there soon.

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.

Suggestion: adjustment for license filename match
2 participants