Skip to content

ci: add a simple check to make sure the image builds for PRs#159

Merged
colisee merged 1 commit intoLibreBooking:masterfrom
JohnVillalovos:jlvillal/pr_ci
Mar 1, 2026
Merged

ci: add a simple check to make sure the image builds for PRs#159
colisee merged 1 commit intoLibreBooking:masterfrom
JohnVillalovos:jlvillal/pr_ci

Conversation

@JohnVillalovos
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 1, 2026 17:51
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a GitHub Actions workflow to build the Docker image on pull_request events, acting as a CI smoke-check that the image still builds for incoming PRs.

Changes:

  • Introduces a new PR-triggered workflow that checks out the repo and runs a Docker build via Buildx.
  • Passes build arguments into the Docker build (PHP/Composer versions and LibreBooking app ref).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment from other PR.

I would prefer instead a build_tag_request GitHub action, for all published images should come from an official librebooking/docker release.

I use my forked GitHub repository colisee/librebooking-docker and my personal docker hub colisee/librebooking for testing purposes.

@colisee colisee merged commit f62cbc5 into LibreBooking:master Mar 1, 2026
1 check passed
@JohnVillalovos JohnVillalovos deleted the jlvillal/pr_ci branch March 1, 2026 19:48
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.

3 participants