Skip to content

Conversation

juliangruber
Copy link
Member

Copy link
Contributor

@pyropy pyropy left a comment

Choose a reason for hiding this comment

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

Looks great!

@bajtos
Copy link
Contributor

bajtos commented Sep 16, 2025

We need to use app-specific token for deploying to Fly.io, see #27 and secrets.FLY_TOKEN_CALIBRATION, secrets.FLY_TOKEN_MAINNET.

@juliangruber
Copy link
Member Author

We need to use app-specific token for deploying to Fly.io, see #27 and secrets.FLY_TOKEN_CALIBRATION, secrets.FLY_TOKEN_MAINNET.

bba38ac

Copy link
Contributor

@bajtos bajtos left a comment

Choose a reason for hiding this comment

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

Looks mostly good 👍🏻

needs: [test]
strategy:
matrix:
network: [calibration, mainnet]
Copy link
Contributor

@bajtos bajtos Sep 16, 2025

Choose a reason for hiding this comment

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

I believe this line is not needed if you use the include block. I think with this line, you will get 4 jobs - two jobs with no FLY_API_TOKEN_NAME (created from the network matrix field) and two jobs from the include block.

Suggested change
network: [calibration, mainnet]

Copy link
Member Author

Choose a reason for hiding this comment

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

@juliangruber juliangruber merged commit 3e442d5 into main Sep 17, 2025
2 checks passed
@juliangruber juliangruber deleted the add/dual-deployment branch September 17, 2025 08:30
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