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

Remove duplicated be param option #384

Merged
merged 2 commits into from
Feb 5, 2020

Conversation

juhoinkinen
Copy link
Member

@juhoinkinen juhoinkinen commented Feb 5, 2020

The option to override BE params was added as a decorator to most CLI commands in #289, but the option was already existing in the optimize command, and I did not notice to remove it at that time. This led only to duplication of the option in the CLI help message.

Also updates the log message about initializing Maui Service tagger to Maui Server, left from #344 .

@juhoinkinen juhoinkinen added the bug label Feb 5, 2020
@juhoinkinen juhoinkinen added this to the 0.46 milestone Feb 5, 2020
@sonarcloud
Copy link

sonarcloud bot commented Feb 5, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@osma osma self-requested a review February 5, 2020 11:10
@juhoinkinen juhoinkinen merged commit 3c3761b into master Feb 5, 2020
@juhoinkinen juhoinkinen deleted the remove-duplicated-be-param-option branch February 5, 2020 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants