-
Notifications
You must be signed in to change notification settings - Fork 508
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
📖 Adding missing documentation for Token-Permissions #1656
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1656 +/- ##
==========================================
+ Coverage 58.50% 61.69% +3.18%
==========================================
Files 58 58
Lines 5801 5801
==========================================
+ Hits 3394 3579 +185
+ Misses 2170 1979 -191
- Partials 237 243 +6 |
Integration tests success for |
Integration tests success for |
Integration tests success for |
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.
Looks good to me. Will also wait for @laurentsimon's approval.
Integration tests success for |
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.
Thanks Chris!
Integration tests success for |
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Docs update
What is the current behavior? (You can also link to an open issue here)
The Token-Permissions check documentation does not include documentation on when the score is reduced due to certain write permissions being present on a job.