ci: add emojis to workflow names and improve their security#4693
ci: add emojis to workflow names and improve their security#4693okineadev wants to merge 4 commits intovuejs:mainfrom
Conversation
|
I'm not in favor of the emojis. It looks cool and all but feels like overdoing it in CI runs.
release-tag.yml doesn't need write access to issues or PRs.
|
Then if I'm not mistaken, there will be no permissions at all
No, it's better to have it in the root of the project, that's how it's done everywhere Is there any reason to store this file in the |
To avoid cluttering the main view.
GitHub allows storing it at
It doesn't need any permission except to create a release. It probably doesn't even need the checkout step. |
Fixed |
c5a8428 to
8abbe29
Compare
084cf43 to
d3a1567
Compare
15a0278 to
09af6c7
Compare
Description
I improved the security of workflows by restricting permissions and credentials, before running
zizmorto check for problemsI also added emojis to the workflow names because they make navigation much better and it's easier to understand what they do
Tip
The author of this PR can publish a preview release by commenting
/publishbelow.