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

feat(api/config): Add Lifecycle types into Config #121

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lshemesh
Copy link

Introducing lifecycle hooks into Deis workflow requires that these types be added to config. These changes will work along side deis/controller#1311 and further changes in deiscli.

Introducing lifecycle hooks into Deis workflow requires
that these types be added to config. This change is required
in order to make further changes to deiscli and in conjunction
with deis/controller#1311
@mboersma mboersma added this to the v2.17 milestone Jul 28, 2017
@mboersma
Copy link
Member

mboersma commented Aug 1, 2017

Jenkins, test this please.

@codecov-io
Copy link

codecov-io commented Aug 1, 2017

Codecov Report

Merging #121 into master will decrease coverage by 2.32%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #121      +/-   ##
==========================================
- Coverage   70.71%   68.38%   -2.33%     
==========================================
  Files          28       28              
  Lines         898     1202     +304     
==========================================
+ Hits          635      822     +187     
- Misses        147      264     +117     
  Partials      116      116
Impacted Files Coverage Δ
api/config.go 84.61% <ø> (ø) ⬆️
certs/certs.go 64.17% <0%> (-5.64%) ⬇️
hooks/hooks.go 57.14% <0%> (-4.77%) ⬇️
keys/keys.go 58.33% <0%> (-4.63%) ⬇️
users/users.go 45.45% <0%> (-4.55%) ⬇️
whitelist/whitelist.go 66.66% <0%> (-3.93%) ⬇️
perms/perms.go 72.72% <0%> (-3.47%) ⬇️
domains/domains.go 62.5% <0%> (-3.02%) ⬇️
tls/tls.go 56.36% <0%> (-2.62%) ⬇️
ps/ps.go 76.81% <0%> (-2.44%) ⬇️
... and 13 more

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 5df54e3...e26593e. Read the comment docs.

@mboersma mboersma modified the milestones: v2.18, v2.17 Aug 8, 2017
@mboersma mboersma removed this from the v2.18 milestone Sep 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants