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

adding the use command and the active projects column to the list com… #28

Merged
merged 6 commits into from
Jun 5, 2019

Conversation

JenniferMah
Copy link
Contributor

@JenniferMah JenniferMah commented Jun 4, 2019

…mand

Fixes

Checklist

  • I have made a material change to the repo (functionality, testing, spelling, grammar)
  • I updated my branch with the master branch.
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation about the functionality in the appropriate .md file
  • I have added in line documentation to the code I modified

Contributing to Twilio

All third party contributors acknowledge that any contributions they provide will be made under the same open source license that the open source project is provided under.

Short description of what this PR does:

  • Adds the use command giving users the ability to specify which project they want to use.
  • Adds an active column that displays which project is active

If you have questions, please file a support ticket, or create a Github Issue in this repository.

@codecov
Copy link

codecov bot commented Jun 4, 2019

Codecov Report

Merging #28 into master will increase coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   98.36%   98.42%   +0.05%     
==========================================
  Files          21       22       +1     
  Lines         551      570      +19     
==========================================
+ Hits          542      561      +19     
  Misses          9        9
Impacted Files Coverage Δ
src/commands/project/use.js 100% <100%> (ø)
src/commands/project/list.js 100% <100%> (ø) ⬆️
src/commands/project/add.js 97.82% <100%> (-0.03%) ⬇️

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 d7db476...4d7750b. Read the comment docs.

test/commands/project/use.test.js Outdated Show resolved Hide resolved
src/commands/project/use.js Outdated Show resolved Hide resolved
test/commands/project/list.test.js Show resolved Hide resolved
Copy link
Contributor

@childish-sambino childish-sambino left a comment

Choose a reason for hiding this comment

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

+1

@childish-sambino childish-sambino merged commit b0a75ca into master Jun 5, 2019
@childish-sambino childish-sambino deleted the UseCommand branch June 5, 2019 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants