CI: link lint + spell check #179
Conversation
github-actions
bot
added
packet-forward-middleware
Reecepbcups
changed the title
| @@ -0,0 +1,5 @@ | |||
| [codespell] | |||
| skip = *.pulsar.go,*.pb.go,*.pb.gw.go,*.cosmos_orm.go,*.json,*.git,*.js,crypto/keys,fuzz,*.h,proto/tendermint,*.bin,*.sum,*.mod,*_test.go | |||
There was a problem hiding this comment.
i notice that here we skip Go test files but your PR in the relayer didn't, is this on purpose?
There was a problem hiding this comment.
yes, many test files use things like userA, step, and other words which are commonly flagged as incorrectly spelled. With the different cases it messes with codespell and is kind of a pain. so just ignoring for now
| 200, | ||
| 206 | ||
| ] | ||
| } |
There was a problem hiding this comment.
why the need for this config file in this repo but not in the relayer repo?
There was a problem hiding this comment.
There are more links in this repo and some edge case URLs being used. Given the higher failure rate the config helps with some flakey URLs (cough cough twittter)
|
If it's not any trouble could we backport this to the other maintained branches? If you don't have any extra bandwidth we could just open an issue as a reminder and get someone else assigned to it at some point. |
closes #178
Fixes all broken links and common spelling issues