Skip to content

Conversation

@itaybre
Copy link
Collaborator

@itaybre itaybre commented Dec 23, 2024

No description provided.

option :api_token, type: :string, required: false,
desc: 'API token for authentication, defaults to ENV[EMERGE_API_TOKEN]'
option :build_id, type: :string, required: true, desc: 'Build ID to download'
option :install, type: :boolean, required: false, desc: 'Install the build on the device'
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: maybe this should default to true?


register 'build-distribution' do |prefix|
prefix.register 'validate-app', Commands::ValidateApp
prefix.register 'download-and-install', Commands::DownloadAndInstall
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe for brevity this can be install? Otherwise emerge build-distribution download-and-install is a big mouthful to type

Base automatically changed from feature/binary_validation_distribution to main December 24, 2024 18:16
@itaybre itaybre merged commit fe06ff3 into main Dec 25, 2024
5 checks passed
@itaybre itaybre deleted the feature/download_install_binary branch December 25, 2024 14:46
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