You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There were two nolint directives added in #19 that mute the linter. These should be removed, or kept, depending on what the upstream cosmos-sdk does about some utility functions that were removed in the tendermint master branch, but kept in the latest release
We have a few options, one of which should eventually become the clear solution, as this linting issue is a tad complex. It's unclear if the linter should or shouldn't be picking up the write permission error. There are many different permissions used, including a few 0777. They seem to be avoiding detection, as the linter misses fucntions that use the tmos package. Ultimately, we should probably do whatever the cosmos-sdk does. For now though, the nolint directive has been added for our own sanity.
The text was updated successfully, but these errors were encountered:
There were two
nolint
directives added in #19 that mute the linter. These should be removed, or kept, depending on what the upstream cosmos-sdk does about some utility functions that were removed in the tendermint master branch, but kept in the latest releaseWe have a few options, one of which should eventually become the clear solution, as this linting issue is a tad complex. It's unclear if the linter should or shouldn't be picking up the write permission error. There are many different permissions used, including a few
0777
. They seem to be avoiding detection, as the linter misses fucntions that use thetmos
package. Ultimately, we should probably do whatever the cosmos-sdk does. For now though, thenolint
directive has been added for our own sanity.The text was updated successfully, but these errors were encountered: