Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update nixpkgs snapshot #294

Merged
merged 1 commit into from
Aug 1, 2024
Merged

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Aug 1, 2024

No description provided.

@purcell purcell closed this Aug 1, 2024
@purcell purcell reopened this Aug 1, 2024
@purcell
Copy link
Owner

purcell commented Aug 1, 2024

@akirak These PRs are created automatically each week, but the Actions don't get run automatically, so I typically close and re-open the PR, then merge once done. We need to make sure the Actions have always run before code lands in master, otherwise cachix might not be fully populated.

@purcell purcell closed this Aug 1, 2024
@purcell purcell reopened this Aug 1, 2024
@akirak
Copy link
Collaborator

akirak commented Aug 1, 2024

These PRs are created automatically each week, but the Actions don't get run automatically, so I typically close and re-open the PR, then merge once done.

Yes, I know. That's one of the workarounds for create-pull-request action to trigger workflow runs. I use another way with PAT in my repositories, but this is your repository, so I will stick with the current way. Thank you for your explanation.

@purcell
Copy link
Owner

purcell commented Aug 1, 2024

I'd be happy to change the process, since this means a manual action each week, just couldn't easily figure it out when I set this up.

@purcell purcell merged commit 69c9d2b into master Aug 1, 2024
101 checks passed
@purcell purcell deleted the nixpkgs-snapshot-update-1722503050 branch August 1, 2024 11:52
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.

2 participants