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.

Sign up for GitHub

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

Jump to bottom

rewrite GH actions #38

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

rewrite GH actions #38

wants to merge 5 commits into from

Conversation

NL-TCH
Copy link
Collaborator

@NL-TCH NL-TCH commented May 3, 2024

NEW:

  • Build the docker image at every new PR (but no push until merge)

CHANGED:

  • Build and push are simplified
  • New image is RaspAP/raspap-docker:latest and RaspAP/raspap-docker:{runID} so in an emergency old image versions can be used

@NL-TCH
Copy link
Collaborator Author

NL-TCH commented May 3, 2024 •
edited

The build at new PR is that we do not build/test until merge but the PR submitter can see if their changes break our image before merging :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@NL-TCH