Skip to content

[18.09] Bump licensing lib#1458

Merged
andrewhsu merged 2 commits intodocker:18.09from
dhiltgen:bump_licensing_lib
Oct 19, 2018
Merged

[18.09] Bump licensing lib#1458
andrewhsu merged 2 commits intodocker:18.09from
dhiltgen:bump_licensing_lib

Conversation

@dhiltgen
Copy link
Contributor

Removes the billing profile flow which is now handled on the back-end.

Removes the billing profile flow which is now handled on the back-end.

Signed-off-by: Daniel Hiltgen <daniel.hiltgen@docker.com>
Signed-off-by: Daniel Hiltgen <daniel.hiltgen@docker.com>
@codecov-io
Copy link

Codecov Report

Merging #1458 into 18.09 will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##            18.09    #1458   +/-   ##
=======================================
  Coverage   54.07%   54.07%           
=======================================
  Files         290      290           
  Lines       19391    19391           
=======================================
  Hits        10485    10485           
  Misses       8237     8237           
  Partials      669      669

@andrewhsu andrewhsu changed the title Bump licensing lib [18.09] Bump licensing lib Oct 18, 2018
Copy link
Contributor

@andrewhsu andrewhsu left a comment

Choose a reason for hiding this comment

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

LGTM

this will help get docker engine activate to work with trial licenses for 18.09.0

@silvin-lubecki
Copy link
Contributor

LGTM

@andrewhsu andrewhsu merged commit e245b72 into docker:18.09 Oct 19, 2018
@thaJeztah thaJeztah added this to the 18.09.0 milestone Oct 22, 2018
@dhiltgen
Copy link
Contributor Author

@andrewhsu I tried to forward port this to master, but it looks like we're a bit behind and it had a merge conflict. Not sure how you want to track that so it doesn't fall through the cracks.

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.

6 participants