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

CLI command name fix + bump shadow plugin version + cleanup #164

Merged
merged 2 commits into from
Oct 26, 2018

Conversation

tovbinm
Copy link
Collaborator

@tovbinm tovbinm commented Oct 25, 2018

Related issues
CLI shows invalid command name.

Describe the proposed solution
Updated the name + shadow plugin version bump

Describe alternatives you've considered
NA

@codecov
Copy link

codecov bot commented Oct 26, 2018

Codecov Report

Merging #164 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #164      +/-   ##
==========================================
- Coverage   85.75%   85.73%   -0.03%     
==========================================
  Files         303      303              
  Lines        9912     9912              
  Branches      543      543              
==========================================
- Hits         8500     8498       -2     
- Misses       1412     1414       +2
Impacted Files Coverage Δ
...in/scala/com/salesforce/op/cli/CommandParser.scala 98.07% <100%> (ø) ⬆️
...es/src/main/scala/com/salesforce/op/OpParams.scala 85.71% <0%> (-4.09%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5a46ac1...153ecc4. Read the comment docs.

@tovbinm
Copy link
Collaborator Author

tovbinm commented Oct 26, 2018

@vpatryshev

Copy link
Collaborator

@leahmcguire leahmcguire left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -49,8 +49,8 @@ configure(allProjs) {
sourceCompatibility = 1.8
targetCompatibility = 1.8

// Max: Set this to the main class for cli.
// Could not figure out how to configure it just for the cli subproject
// Set this to the main class for cli.
Copy link
Collaborator

Choose a reason for hiding this comment

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

:-)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

There is currently no other way. I also checked ;)

@tovbinm tovbinm merged commit 4572d38 into master Oct 26, 2018
@tovbinm tovbinm deleted the mt/shadow2 branch October 26, 2018 19:34
@tovbinm tovbinm restored the mt/shadow2 branch October 26, 2018 19:34
@tovbinm tovbinm deleted the mt/shadow2 branch October 26, 2018 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants