Skip to content

Broaden the regex in case the ©, (c), or nothing follows 'Copyright' - #1884

Merged
tegioz merged 1 commit into
mainfrom
update-copyright-regex
Mar 13, 2026
Merged

tegioz merged 1 commit into
mainfrom
update-copyright-regex

Conversation

@krook

@krook krook commented Mar 12, 2026

Copy link
Copy Markdown
Member

Technically "Copyright ©" is redundant, so the "©" is sometimes dropped in footers. And in other ASCII places "(c)" or "(C)" are used. This allows for them all.

Signed-off-by: Daniel Krook <krook@linux.com>
@krook
krook requested a review from cynthia-sg March 12, 2026 20:32

@tegioz tegioz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @krook 👍

@tegioz
tegioz added this pull request to the merge queue Mar 13, 2026
Merged via the queue into main with commit 9b86bd3 Mar 13, 2026
6 checks passed
@tegioz
tegioz deleted the update-copyright-regex branch March 13, 2026 10:16
@krook

krook commented Mar 30, 2026

Copy link
Copy Markdown
Member Author

Thanks @tegioz. I see that this has been merged, but it hasn't been updated on the site here: https://clomonitor.io/docs/topics/checks/#legal

Did I miss a file?

@tegioz

tegioz commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

We haven't deployed it yet @krook, we'll do it this week 🙂

@krook

krook commented Mar 30, 2026

Copy link
Copy Markdown
Member Author

Ah, Ok, thank you!

@tegioz

tegioz commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

Deployed @krook. You should see it updated within the next day (or earlier, if there are changes in the repository).

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