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

Updated Consul Download URLs #252

Merged
merged 3 commits into from
Dec 11, 2015
Merged

Updated Consul Download URLs #252

merged 3 commits into from
Dec 11, 2015

Conversation

Ginja
Copy link
Contributor

@Ginja Ginja commented Dec 10, 2015

Looks like consul is now prefixing their download URLs with 'consul'. Updated appropriate attributes, and binary_filename methods.

Reference - https://releases.hashicorp.com/consul/0.6.0/

Looks like consul is now prefixing their download URLs with 'consul'.
Updated appropriate attributes, and binary_filename methods.
Reference - https://releases.hashicorp.com/consul/0.6.0/
Now using the new Consul binary URL scheme
@codecov-io
Copy link

Current coverage is 72.56%

Merging #252 into master will not affect coverage as of bdbee0d

@@            master    #252   diff @@
======================================
  Files            6       6       
  Stmts          328     328       
  Branches         0       0       
  Methods          0       0       
======================================
  Hit            238     238       
  Partial          0       0       
  Missed          90      90       

Review entire Coverage Diff as of bdbee0d

Powered by Codecov. Updated on successful CI builds.

@scalp42
Copy link
Contributor

scalp42 commented Dec 11, 2015

@johnbellone any chance to look at it ?

johnbellone added a commit that referenced this pull request Dec 11, 2015
@johnbellone johnbellone merged commit ede4642 into sous-chefs:master Dec 11, 2015
@Ginja Ginja mentioned this pull request Dec 11, 2015
@knightorc knightorc deleted the update-consul-webui-url branch June 6, 2018 23:05
@lock
Copy link

lock bot commented Apr 25, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 25, 2020
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.

4 participants