-
-
Notifications
You must be signed in to change notification settings - Fork 32
test(ci): add github token minimal permissions in all workflows #624
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
david-allison
left a comment
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.
Sure! Can we do a test publish after this goes through
|
Definitely +1 on test publish, will merge dependencies prior and set a new internal version |
bee19af to
cd0eefd
Compare
|
That's annoying, they were running full / expected speed it seems
|
|
1- successful publish! https://repo1.maven.org/maven2/io/github/david-allison/anki-android-backend/0.1.63-anki25.09.2/
Okay, there is still apparently some warning here, I'll look into that - it was running green throughout the day 🤔 |
|
Ah, it's generated rust code - going to ignore for now but could use a preen later if someone wanted something unimportant to pick at that had no real world purpose ("tuning github actions codeql scanning to ignore generated code" ? meh) |

Should fix CodeQL code scanning alerts
Appears to me that the workflows only need
contents: readfor the github token, but I suppose we'll see