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 facter dependency #236

Merged
merged 1 commit into from
Apr 2, 2020
Merged

Conversation

swrobel
Copy link
Collaborator

@swrobel swrobel commented Mar 31, 2020

  • Facter required a native extension, and was used to determine Linux distro - use /etc/os-release to get that info much more simply
  • Add a new spec to test custom distro detection
  • Remove Mint distro as it was just a direct descendant of Ubuntu with no overrides
  • Remove dependency on git from gemspec - use pure ruby to get file lists instead
  • Use docker on Travis CI to test all supported distros (only testing distro detection for now)
  • Only run full spec suite using latest ruby

* Facter required a native extension, and was used to determine Linux distro - use /etc/os-release to get that info much more simply
* Add a new spec to test custom distro detection
* Remove Mint distro as it was just a direct descendant of Ubuntu with no overrides
* Remove dependency on git from gemspec - use pure ruby to get file lists instead
* Use docker on Travis CI to test all supported distros (only testing distro detection for now)
* Only run full spec suite using latest ruby
@swrobel swrobel merged commit 12ad468 into codemancers:master Apr 2, 2020
@swrobel swrobel deleted the remove-facter branch April 2, 2020 15:04
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.

None yet

1 participant