Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Allow JSON output from clients:create and authorizations:create #30

Merged
merged 2 commits into from
May 19, 2017

Conversation

ojacobson
Copy link
Contributor

Sometimes you need more than just the token, in a parseable form. I'm using it like this:

heroku authorizations:create \
  --description "Example app dev env: ${HEROKU_APP}" \
  --scope identity \
  --expires-in $((3 * 24 * 60 * 60)) \
  --json |
  jq -r '.id + " " + .access_token.token' |
  read HEROKU_OAUTH_ID HEROKU_OAUTH_SECRET

@codecov-io
Copy link

codecov-io commented May 19, 2017

Codecov Report

Merging #30 into master will decrease coverage by 1.42%.
The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
- Coverage   99.23%   97.81%   -1.43%     
==========================================
  Files          15       15              
  Lines         130      137       +7     
==========================================
+ Hits          129      134       +5     
- Misses          1        3       +2
Impacted Files Coverage Δ
commands/clients/create.js 85.71% <80%> (-5.96%) ⬇️
commands/authorizations/create.js 92.3% <88.88%> (-7.7%) ⬇️

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 4d4ff95...dd38cbe. Read the comment docs.

@jdx jdx merged commit 9a82211 into master May 19, 2017
@jdx jdx deleted the json-create branch May 19, 2017 21:33
jdx added a commit that referenced this pull request Aug 28, 2017
Allow JSON output from clients:create and authorizations:create
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants