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.

1 participant