Fix various documentation, user-facing, and source comment typos #16367

Merged
luzpaz merged 4 commits from typos into main 2021-07-08 11:38:13 +00:00
luzpaz commented 2021-07-07 20:08:48 +00:00 (Migrated from github.com)

Found using codespell -q 3 -S ./options/locale,./vendor -L ba,pullrequest,pullrequests,readby

Found using `codespell -q 3 -S ./options/locale,./vendor -L ba,pullrequest,pullrequests,readby`
techknowlogick reviewed 2021-07-07 20:43:49 +00:00

Thanks for this PR :) The only change I am unsure of, is this one as this is copied directly from the upstream fomantic library

Thanks for this PR :) The only change I am unsure of, is this one as this is copied directly from the upstream fomantic library
jpraet reviewed 2021-07-07 21:13:37 +00:00
Contributor

Indeed, on line 351 we also have successful. So there will be a naming clash now.
So I think it should stay intentionally misspelled as succesful, or maybe it can be removed?

Indeed, on line 351 we also have `successful`. So there will be a naming clash now. So I think it should stay intentionally misspelled as `succesful`, or maybe it can be removed?
techknowlogick reviewed 2021-07-07 21:55:55 +00:00

I'd prefer it to stay intentionally misspelled, at least as long as it remains upstream.

I'd prefer it to stay intentionally misspelled, at least as long as it remains upstream.
luzpaz (Migrated from github.com) reviewed 2021-07-07 22:07:14 +00:00
luzpaz (Migrated from github.com) commented 2021-07-07 22:07:14 +00:00

Should I revert all the upstream fomantic changes? or just this one?

Should I revert all the upstream fomantic changes? or just this one?
techknowlogick reviewed 2021-07-07 22:44:33 +00:00

looking at the other fomantic ones, they are comments/strings but this one is the name of the function so I think reverting just this one is ok.

looking at the other fomantic ones, they are comments/strings but this one is the name of the function so I think reverting just this one is ok.
luzpaz (Migrated from github.com) reviewed 2021-07-08 01:19:01 +00:00
luzpaz (Migrated from github.com) commented 2021-07-08 01:19:00 +00:00

Done 88f8829

Done 88f8829
techknowlogick reviewed 2021-07-08 03:00:37 +00:00

awesome. thanks again :)

awesome. thanks again :)
techknowlogick approved these changes 2021-07-08 03:00:47 +00:00
6543 (Migrated from github.com) reviewed 2021-07-08 10:26:25 +00:00
6543 (Migrated from github.com) commented 2021-07-08 10:26:24 +00:00
	// the environment is set by serv command
```suggestion // the environment is set by serv command ```
6543 (Migrated from github.com) reviewed 2021-07-08 10:26:42 +00:00
6543 (Migrated from github.com) commented 2021-07-08 10:26:42 +00:00
	// the environment is set by serv command
```suggestion // the environment is set by serv command ```
6543 (Migrated from github.com) approved these changes 2021-07-08 10:34:14 +00:00
This repo is archived. You cannot comment on pull requests.
No reviewers
No Milestone
No project
No Assignees
3 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: lunny/gitea#16367
No description provided.