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

Use HTTPS for yumrepo when applicable #351

Merged

Conversation

Projects
None yet
2 participants
@aknarts
Copy link
Contributor

commented Aug 29, 2016

Updated version of #330 using the newer way of deciding which protocol to use

@olivielpeau olivielpeau added this to the 2.6.0 milestone Sep 7, 2016

@olivielpeau

This comment has been minimized.

Copy link
Member

commented Sep 9, 2016

Thanks for the updated PR @aknarts

After giving some thought I don't see how it could break BC, so let's merge this for the next minor release even though it slightly changes the default value of an attribute.

@olivielpeau olivielpeau merged commit de79509 into DataDog:master Sep 9, 2016

2 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.