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

Remove php::repo::ubuntu::ppa #218

Merged
merged 3 commits into from
Jun 23, 2016

Conversation

tdm4
Copy link
Contributor

@tdm4 tdm4 commented Jun 23, 2016

Removes php::repo::ubuntu::ppa (Issue #214 )
Also removes reference to it in README.md

@tdm4 tdm4 changed the title Remove php::repo::ubuntu::ppa #214 Remove php::repo::ubuntu::ppa Jun 23, 2016
@tdm4
Copy link
Contributor Author

tdm4 commented Jun 23, 2016

@mmoll Can you tell it to re-run the spec tests? They come out as fine (no errors) on my system. I think the Travis-CI system got confused with 2 commits in a short amount of time.

@mmoll
Copy link
Contributor

mmoll commented Jun 23, 2016

I'll re-run them soon, as voxpupuli/puppet-lint-absolute_classname-check#6 will be resolved in the next hour or so.

@tdm4
Copy link
Contributor Author

tdm4 commented Jun 23, 2016

@mmoll I guess they eventually ran on their own!

@mmoll
Copy link
Contributor

mmoll commented Jun 23, 2016

Nope, I did restart them 🚒
This change is fine, merging...
Thanks again @tdm4!

@mmoll mmoll merged commit ace58db into voxpupuli:master Jun 23, 2016
@tdm4
Copy link
Contributor Author

tdm4 commented Jun 23, 2016

Great stuff! Thanks @mmoll !

@tdm4 tdm4 deleted the Remove_php_repo_ubuntu_ppa branch June 23, 2016 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants