Skip to content
New issue

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

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

Already on GitHub? # to your account

feature(ci): Run docker build on all pushes #1643

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

sgillespie
Copy link
Contributor

Description

Add your description here, if it fixes a particular issue please provide a link to the issue.

Checklist

  • Commit sequence broadly makes sense
  • Commits have useful messages
  • New tests are added if needed and existing tests are updated
  • Any changes are noted in the changelog
  • Code is formatted with fourmolu on version 0.10.1.0 (which can be run with scripts/fourmolize.sh)
  • Self-reviewed the diff

Migrations

  • The pr causes a breaking change of type a,b or c
  • If there is a breaking change, the pr includes a database migration and/or a fix process for old values, so that upgrade is possible
  • Resyncing and running the migrations provided will result in the same database semantically

If there is a breaking change, especially a big one, please add a justification here. Please elaborate
more what the migration achieves, what it cannot achieve or why a migration is not possible.

@sgillespie sgillespie force-pushed the feature/ci-release-tasks branch 3 times, most recently from 08a5401 to f632796 Compare March 5, 2024 19:48
@sgillespie sgillespie force-pushed the feature/ci-release-tasks branch 2 times, most recently from 13aa885 to 8b4da85 Compare May 22, 2024 17:55
The `pull_request` trigger tries to merge the base branch, and
Hydra uses the commit tip, so nix will usually not resolve the
correct artifact.
@sgillespie sgillespie force-pushed the feature/ci-release-tasks branch from 8b4da85 to e9ee97d Compare May 23, 2024 20:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant