Skip to content

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