-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Fix various typos #18219
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
Fix various typos #18219
Conversation
Some of these typos are dupes of #18215 |
👍 let's wait for #18215 |
@@ -6,4 +6,4 @@ Noweb is available free for any use in any field of endeavor. You may redistribu | |||
|
|||
You may sell noweb if you wish. For example, you may sell a CD-ROM including noweb. | |||
|
|||
You may sell a derived work, provided that all source code for your derived work is available, at no additional charge, to anyone who buys your derived work in any form. You must give permisson for said source code to be used and modified under the terms of this license. You must state clearly that your work uses or is based on noweb and that noweb is available free of change. You must also request that bug reports on your work be reported to you. | |||
You may sell a derived work, provided that all source code for your derived work is available, at no additional charge, to anyone who buys your derived work in any form. You must give permission for said source code to be used and modified under the terms of this license. You must state clearly that your work uses or is based on noweb and that noweb is available free of change. You must also request that bug reports on your work be reported to you. |
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.
A typo in a LICENSE 😆
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.
Upstream certainly has the typo too: https://fedoraproject.org/wiki/Licensing:Noweb?rd=Licensing/Noweb, not sure if this is the definitive source.
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.
Does anyone want to make a patch and submit it upstream, or shall I do it?
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.
@luzpaz please try to patch it upstream - and revert the change in our copy, as it will be overwritten anyway (by our tool chain)
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.
Then we might not need to wait for the upstream, and now merge other fixes in this PR first.
Found via `codespell -q 3 -S ./options/locale,./vendor -L ba,pullrequest,pullrequests,readby,te,unknwon`
Revert `modules/git/diff_test.go` revision
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.
LGTM. If upstream has new fixes, we can have them in future.
Follow-up to go-gitea#18219
* 'main' of https://github.com/go-gitea/gitea: [skip ci] Updated translations via Crowdin Fix source typos (go-gitea#18227) Fix various typos (go-gitea#18219) Remove `ioutil` (go-gitea#18222) [skip ci] Updated translations via Crowdin
Found via `codespell -q 3 -S ./options/locale,./vendor -L ba,pullrequest,pullrequests,readby,te,unknwon` Co-authored-by: Lunny Xiao <[email protected]>
Follow-up to go-gitea#18219
Found via
codespell -q 3 -S ./options/locale,./vendor -L ba,pullrequest,pullrequests,readby,te,unknwon