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: flyctl deploy flag from region to regions #56

Merged
merged 1 commit into from
May 6, 2024

Conversation

bryantbrock
Copy link
Contributor

@bryantbrock bryantbrock commented May 4, 2024

flyctl v0.2.47 [PR] silently broke everyones CI/CDs. It broke my review apps. This fixes that change.

This really needs to go out ASAP as it breaks ALL review apps until it's fixed. I'm changing mine to manually run this code until this is merged in and released.

@bryantbrock
Copy link
Contributor Author

ETA on getting merged in?

@ragbadaskar
Copy link

ragbadaskar commented May 6, 2024

@bryantbrock how do we manage the situation till this PR is merged? Can we use the edits your made in the steps?

@Liinkiing
Copy link

@bryantbrock how do we manage the situation till this PR is merged? Can we use the edits your made in the steps?

Yes you can, that's what I'm doing (thanks @bryantbrock)

In your workflow, you can update the uses to

        uses: bryantbrock/fly-pr-review-apps@6e34c6a6405b85220fb22ee564d32ce89c3dc384

@jsierles jsierles merged commit 6f79ec3 into superfly:main May 6, 2024
dennisrb2 added a commit to rb2-bv/fly-pr-review-apps that referenced this pull request May 6, 2024
fix: flyctl deploy flag from region to regions (superfly#56)
lbrito1 added a commit to lbrito1/fly-pr-review-apps that referenced this pull request May 6, 2024
lbrito1 added a commit to lbrito1/fly-pr-review-apps that referenced this pull request May 6, 2024
@ragbadaskar
Copy link

ragbadaskar commented May 7, 2024

Thanks for the merge. I'm trying to push preview app but fails with the same error.
uses: superfly/[email protected]

There is no version change. So what am I missing?

Thank you @bryantbrock for the patch.

@clouvet
Copy link

clouvet commented May 7, 2024

@ragbadaskar you should see a new release now, v1.2.1.

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.

6 participants