Skip to content

workflows/codeowners: Dry mode for now#347538

Merged
ambroisie merged 1 commit intoNixOS:masterfrom
tweag:codeowners-actual-dry-mode
Oct 9, 2024
Merged

workflows/codeowners: Dry mode for now#347538
ambroisie merged 1 commit intoNixOS:masterfrom
tweag:codeowners-actual-dry-mode

Conversation

@infinisil
Copy link
Member

Apparently it started requesting reviews from code owners already because the DRY_MODE from the global env was overridden in the local job declaration: #347354 (comment)

The intention when introduced in #336261 was to still run it in full dry mode for now.

This change fixes the behavior, making sure it always runs in dry mode.


Add a 👍 reaction to pull requests you find important.

Apparently it started requesting reviews from code owners already
because the DRY_MODE from the global env was overridden in the local job
declaration: NixOS#347354 (comment)
@infinisil infinisil requested a review from philiptaron October 9, 2024 16:37
@github-actions github-actions bot added the 6.topic: policy discussion Discuss policies to work in and around Nixpkgs label Oct 9, 2024
@infinisil
Copy link
Member Author

infinisil commented Oct 9, 2024

Merging this to avoid more reviews getting requested while figuring things out (there's a bit of a bug with it)

@ambroisie ambroisie merged commit 40e34c4 into NixOS:master Oct 9, 2024
@infinisil infinisil deleted the codeowners-actual-dry-mode branch October 9, 2024 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: policy discussion Discuss policies to work in and around Nixpkgs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants