Skip to content

CI: restrict github app token - #24

Merged
mgovers merged 5 commits into
mainfrom
pgm/feature/restrict-github-app-token
Jul 27, 2026
Merged

CI: restrict github app token#24
mgovers merged 5 commits into
mainfrom
pgm/feature/restrict-github-app-token

Conversation

@mgovers mgovers added improvement Improvement on internal implementation github_actions Pull requests that update GitHub Actions code labels Jul 23, 2026
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>

restrict permissions

Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
nitbharambe
nitbharambe previously approved these changes Jul 24, 2026
@nitbharambe

Copy link
Copy Markdown
Member
image What might this relate to ?

@nitbharambe
nitbharambe dismissed their stale review July 24, 2026 08:38

A question

@mgovers

mgovers commented Jul 24, 2026

Copy link
Copy Markdown
Member Author

image What might this relate to ?

whoops, good catch. I'll go by all PRs. Please hold off on merging for now

@mgovers mgovers added the do-not-merge This should not be merged label Jul 24, 2026
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
figueroa1395
figueroa1395 previously approved these changes Jul 24, 2026
@figueroa1395

Copy link
Copy Markdown
Member

@mgovers after approving I just noticed do-not-merge label doesn't work as intended.

Merging delayed for Monday.

power-grid-model-ci-bot Bot and others added 2 commits July 25, 2026 03:02
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sync version of upstream repository
@mgovers
mgovers merged commit 43e21d2 into main Jul 27, 2026
2 checks passed
@mgovers
mgovers deleted the pgm/feature/restrict-github-app-token branch July 27, 2026 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge This should not be merged github_actions Pull requests that update GitHub Actions code improvement Improvement on internal implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants