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

fix: prevent deploy to production on open PR #11

Merged
merged 1 commit into from
Apr 4, 2024
Merged

fix: prevent deploy to production on open PR #11

merged 1 commit into from
Apr 4, 2024

Conversation

EresDev
Copy link
Contributor

@EresDev EresDev commented Apr 2, 2024

Resolves #8

QA

It does not affect the normal operations of a fork. A fork's change on the development branch (or whatever the default branch is) gets deployed to production. All other branches get deployed to preview.
Action: https://github.com/EresDevOrg/3rd-fork-pay.ubq.fi/actions/runs/8526016642/job/23355077672
image

When a fork opens a pull request to the ubiquity's development branch, it gets deployed to Cloudflare preview.
Action: https://github.com/EresDevOrg/pay.ubq.fi/actions/runs/8526313017/job/23355279324
PR: https://github.com/EresDevOrg/pay.ubq.fi/pull/14
image

When the pull request is merged, it gets deployed to the production.
Action: https://github.com/EresDevOrg/pay.ubq.fi/actions/runs/8526347581/job/23355387087
PR: https://github.com/EresDevOrg/pay.ubq.fi/pull/14
image

@EresDev EresDev self-assigned this Apr 2, 2024
@EresDev EresDev requested review from 0x4007 and rndquu April 2, 2024 16:49
@0x4007
Copy link
Member

0x4007 commented Apr 4, 2024

Looks like @rndquu and I are busy with other urgent projects. This also seems pretty urgent so we can test in production?

@ubiquity/software-development heads up if things break

@0x4007 0x4007 merged commit 5454d2d into ubiquity:main Apr 4, 2024
@ubiquibot ubiquibot bot mentioned this pull request Apr 4, 2024
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.

Deploy to Production from PR
2 participants