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(cli): signing type option issue #6716

Merged
merged 15 commits into from
Aug 3, 2023
Merged

fix(cli): signing type option issue #6716

merged 15 commits into from
Aug 3, 2023

Conversation

liuwin7
Copy link
Contributor

@liuwin7 liuwin7 commented Jul 6, 2023

There are two revision.
The first one is to correct the commander use case, if a option is named with b -, it will change to the Camel Case.
The second is to load the all android build options, not only the options for apksigner .

cli/src/index.ts Outdated Show resolved Hide resolved
Copy link
Member

@markemer markemer left a comment

Choose a reason for hiding this comment

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

Just a few questions, but I think we can make this change.

@markemer markemer changed the title Fix CLI Command [build android] fix(cli): signing type option issue Jul 12, 2023
@liuwin7 liuwin7 requested a review from markemer July 14, 2023 06:04
@liuwin7

This comment was marked as abuse.

@dallastjames dallastjames added the type: bug A confirmed bug report label Jul 31, 2023
@IT-MikeS IT-MikeS merged commit ee0f745 into ionic-team:main Aug 3, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A confirmed bug report
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants