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

feat: add deploy alias #597

Merged
merged 3 commits into from
Oct 28, 2024
Merged

feat: add deploy alias #597

merged 3 commits into from
Oct 28, 2024

Conversation

raaymax
Copy link
Collaborator

@raaymax raaymax commented Oct 15, 2024

No description provided.

@ramedina86
Copy link
Collaborator

@raaymax any ideas on why the CI might be failing?

@raaymax
Copy link
Collaborator Author

raaymax commented Oct 18, 2024

No idea, it's failing the same way also in the other PR..
Maybe the problem is with dev branch?

@raaymax
Copy link
Collaborator Author

raaymax commented Oct 18, 2024

Found this microsoft/playwright#30368
Seems to be related to ubuntu 24 🤔

@ramedina86
Copy link
Collaborator

It does, so how do we fix? cc @FabienArcellier

@ramedina86
Copy link
Collaborator

Looks like it run this time though?

src/writer/command_line.py Outdated Show resolved Hide resolved
prompt="Enter your API key",
hide_input=True, help="Writer API key"
)
@click.option('--env', '-e', multiple=True, default=[], help="Environment to deploy the app to")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe Environments as it can be multiple

Suggested change
@click.option('--env', '-e', multiple=True, default=[], help="Environment to deploy the app to")
@click.option('--env', '-e', multiple=True, default=[], help="Environments to deploy the app to")

Copy link
Collaborator

Choose a reason for hiding this comment

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

Aren't these environment variables?

@raaymax
Copy link
Collaborator Author

raaymax commented Oct 21, 2024

You have no idea how much time I spent in click docs searching for exactly something like this...
Shout out to @FabienArcellier :D

src/writer/deploy.py Outdated Show resolved Hide resolved
Co-authored-by: Ramiro Medina <[email protected]>
@ramedina86 ramedina86 merged commit 1e1da6f into dev Oct 28, 2024
18 checks passed
@ramedina86 ramedina86 deleted the deploy_alias branch December 3, 2024 16:40
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.

4 participants