Skip to content

Conversation

kylecarbs
Copy link
Member

No description provided.

@kylecarbs kylecarbs requested a review from hugodutka October 17, 2025 19:36
@kylecarbs kylecarbs self-assigned this Oct 17, 2025
Copy link
Collaborator

@hugodutka hugodutka left a comment

Choose a reason for hiding this comment

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

I think we should require a blink deploy --config $path_to_config_json flag in CI mode. The .blink directory is in .gitignore by default and it's a hassle to add just .blink/config.json to version control. It'd also allow people to deploy to another agent for staging.

Comment on lines +94 to +97
throw new Error(
"Multiple organizations found. In CI mode, you must first deploy locally to select an organization, " +
"or set the organization in .blink/config.json"
);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
throw new Error(
"Multiple organizations found. In CI mode, you must first deploy locally to select an organization, " +
"or set the organization in .blink/config.json"
);
throw new Error(
"Multiple organizations found. To use CI mode, please deploy in interactive mode first to select an organization and generate .blink/config.json"
);

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.

2 participants