Skip to content

.github: Add job names and restrict permission & credentials #87

Merged
kartben merged 3 commits into
zephyrproject-rtos:mainfrom
str4t0m:actions_improve
Oct 16, 2025
Merged

.github: Add job names and restrict permission & credentials #87
kartben merged 3 commits into
zephyrproject-rtos:mainfrom
str4t0m:actions_improve

Conversation

@str4t0m
Copy link
Copy Markdown
Contributor

@str4t0m str4t0m commented Aug 2, 2025

I reviewed the workflows and noticed that we could set the workflow permission to contents read and avoid the default behavior of the auth token being persisted in the local git confg for subsequent steps of the git checkout action.
Additionally I added a few names to jobs that didn't have any yet.

@str4t0m str4t0m marked this pull request as ready for review August 2, 2025 19:04
Comment thread .github/workflows/docs.yml Outdated
str4t0m added 3 commits August 3, 2025 10:33
Restrict permissions explicitly

Signed-off-by: Thomas Stranger <thomas.stranger@outlook.com>
Restrict default permission of the build workflows.

Signed-off-by: Thomas Stranger <thomas.stranger@outlook.com>
Add names to the jobs

Signed-off-by: Thomas Stranger <thomas.stranger@outlook.com>
@str4t0m str4t0m requested a review from stephanosio August 3, 2025 08:52
@kartben kartben merged commit a60f889 into zephyrproject-rtos:main Oct 16, 2025
6 checks passed
@kartben
Copy link
Copy Markdown
Member

kartben commented Oct 16, 2025

Thanks very much @str4t0m ! Not sure what took us so long to merge this - apologies for that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants