Skip to content

provision/ec2: colourize and emojify ec2 provisioning experience#606

Merged
bobheadxi merged 4 commits into
masterfrom
provision/#597-ec2-output
Mar 17, 2019
Merged

provision/ec2: colourize and emojify ec2 provisioning experience#606
bobheadxi merged 4 commits into
masterfrom
provision/#597-ec2-output

Conversation

@bobheadxi

Copy link
Copy Markdown
Member

🎟️ Ticket(s): Closes #597


👷 Changes

  • new Fprintf in the utils/out API
  • all input prompts now print with cyan colour
  • bling for ec2 provisioning

🔦 Testing Instructions

make cli
./inertia provision ec2 ....

@bobheadxi bobheadxi added the :provision instance provisioning features label Mar 16, 2019
@bobheadxi bobheadxi requested review from a team and yaoharry March 16, 2019 02:44
@bobheadxi bobheadxi added pr: finalized needs review and final approval and removed :provision instance provisioning features labels Mar 16, 2019
@ghost ghost requested review from mRabitsky and terryz21 March 16, 2019 02:45
@codecov

codecov Bot commented Mar 16, 2019

Copy link
Copy Markdown

Codecov Report

Merging #606 into master will decrease coverage by 0.19%.
The diff coverage is 5.27%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #606      +/-   ##
==========================================
- Coverage   56.51%   56.33%   -0.18%     
==========================================
  Files          68       68              
  Lines        3288     3299      +11     
==========================================
  Hits         1858     1858              
- Misses       1194     1205      +11     
  Partials      236      236
Impacted Files Coverage Δ
cmd/core/utils/out/decor.go 86.8% <0%> (-9.04%) ⬇️
provision/ec2.go 7.66% <0%> (-0.22%) ⬇️
cmd/core/utils/input/input.go 76.82% <100%> (ø) ⬆️

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 e96e535...566c9d8. Read the comment docs.

@bobheadxi bobheadxi merged commit 78a1c4b into master Mar 17, 2019
@bobheadxi bobheadxi deleted the provision/#597-ec2-output branch March 17, 2019 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: finalized needs review and final approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants