Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

feature/decorators #57

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

IKatsuba
Copy link

@IKatsuba IKatsuba commented Apr 5, 2019

Added decorators for key parameters of the Command class. This improves auto-completion, and also guarantees correct type checking when compiling TS.

Added decorators for key parameters of the class Command
@salesforce-cla
Copy link

salesforce-cla bot commented Apr 5, 2019

Thanks for the contribution! Before we can merge this, we need @IKatsuba to sign the Salesforce.com Contributor License Agreement.

@codecov
Copy link

codecov bot commented Apr 5, 2019

Codecov Report

Merging #57 into master will increase coverage by 69.10%.
The diff coverage is 87.50%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #57       +/-   ##
===========================================
+ Coverage    0.00%   69.10%   +69.10%     
===========================================
  Files           1        6        +5     
  Lines          10      178      +168     
  Branches        2       35       +33     
===========================================
+ Hits            0      123      +123     
- Misses         10       35       +25     
- Partials        0       20       +20     
Impacted Files Coverage Δ
src/decorators.ts 86.66% <86.66%> (ø)
src/index.ts 78.94% <100.00%> (+78.94%) ⬆️
src/main.ts 61.53% <0.00%> (ø)
src/util.ts 38.88% <0.00%> (ø)
src/command.ts 76.25% <0.00%> (ø)
src/flags.ts 55.00% <0.00%> (ø)

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 b018bb6...3562386. Read the comment docs.

@IKatsuba
Copy link
Author

Any update on this?

@IKatsuba IKatsuba requested a review from jdx January 29, 2020 13:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant