-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Add release for android OS #905
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
Conversation
.golangci.yml
Outdated
@@ -44,7 +44,6 @@ linters-settings: | |||
- dupImport # https://github.com/go-critic/go-critic/issues/845 | |||
- ifElseChain | |||
- octalLiteral | |||
- whyNoLint |
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.
Unfortunately, this is needed for latest v1.22.x releases due to go-critic update. The GitHub check should pass once go-critic/go-critic#845 is merged and deployed.
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.
Any suggestion what should I do now?
- Return the
whyNoLint
config - ???
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.
Just undo the whyNoLint
removal commit and it should be good to merge.
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.
GolangCI failure is expected, yes.
To undo this commit you can do the following:
$ git reset --hard 6fd38300fbdc897e6786e899717e712cbecde0b8
$ git push -f uudashr
Where 6fd3830 is your first commit, and uudashr
is your fork remote name.
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.
Reverted
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.
Hi @tpounds already undo the whyNoLint
, should good now
thank you! |
I've needed to revert this change because a new release can't be built on android. I got errors like these:
|
@jirfag AFAICT, we need to have |
Somehow go get also failed on android
|
Add release for android. Some of use Termux on Android as alternative development machine.