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
(MODULES-2207) Modulesync #121
(MODULES-2207) Modulesync #121
Conversation
gem 'puppet-lint', :require => false | ||
gem 'puppetlabs_spec_helper', :require => false | ||
gem 'puppet_facts', :require => false | ||
gem 'mocha', '~>0.10.5', :require => false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cyberious can you review these changes?
I have pulled out
- nokogiri
- mime-types
- simplecov
- yard
- pry
- rspec-puppet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added back in simplecov
and yard
. May need to add back in rspec-puppet
as well.
This is likely to fail on v4 - it did with Travis CI |
@cyberious This is probably going to be an effort - do you have any quick ideas into why it fails on future parser/ v4.0? |
Determining that the effort here for fixing the failures will be addressed with a new ticket. |
Rebased with fixes from #122 |
This brings the following items under modulesync: * .gitignore * .travis.yml * Gemfile * appveyor.yml * spec/spec.opts More importantly, this ensures that acl has up to date Gemfile, Travis CI and AppVeyor configurations that allow it to be tested properly.
This brings the following items under modulesync:
More importantly, this ensures that acl has up to date Gemfile, Travis
CI and AppVeyor configurations that allow it to be tested properly.