Skip to content

Base branch trigger test#83

Closed
infinisil wants to merge 1 commit intoby-name-base-trigger-stagingfrom
by-name-base-trigger-test
Closed

Base branch trigger test#83
infinisil wants to merge 1 commit intoby-name-base-trigger-stagingfrom
by-name-base-trigger-test

Conversation

@infinisil
Copy link
Copy Markdown
Member

This tests NixOS#282707

@infinisil
Copy link
Copy Markdown
Member Author

infinisil commented Jan 22, 2024

CI succeeds https://github.com/tweag/nixpkgs/actions/runs/7618252980/job/20749017432?pr=83

Now rebasing on a staging mock..

@infinisil infinisil force-pushed the by-name-base-trigger-test branch from 764bf7c to ee238b3 Compare January 22, 2024 22:37
@infinisil
Copy link
Copy Markdown
Member Author

infinisil commented Jan 22, 2024

As expected, CI fails. The base branch is still by-name-base-trigger, which doesn't include the commit from the staging mock branch that adds a new package not via pkgs/by-name: https://github.com/tweag/nixpkgs/actions/runs/7618273215/job/20749082632?pr=83

Now changing the base branch..

@infinisil infinisil changed the base branch from by-name-base-trigger to by-name-base-trigger-staging January 22, 2024 22:39
@infinisil
Copy link
Copy Markdown
Member Author

Can confirm that CI was triggered just from the base branch change and now succeeds: https://github.com/tweag/nixpkgs/actions/runs/7618290690/job/20749138247?pr=83

@infinisil infinisil closed this Jan 25, 2024
@infinisil infinisil deleted the by-name-base-trigger-test branch January 25, 2024 19:40
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.

1 participant