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

aknarts
Copy link
Contributor

@aknarts aknarts 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
Copy link
Member

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
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.

2 participants