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

(maint) Convert module for PDK Compatibility #93

Merged
merged 1 commit into from
Dec 15, 2017

Conversation

bmjen
Copy link
Contributor

@bmjen bmjen commented Dec 15, 2017

No description provided.

@bmjen bmjen force-pushed the pdk-convert branch 4 times, most recently from 03f5115 to e02b7a8 Compare December 15, 2017 13:56
@willmeek
Copy link

willmeek commented Dec 15, 2017

Looks good, but is there a way of ignoring modulesync files?
(These usually begin with "#This file is generated by ModuleSync, do not edit." or otherwise live / templated at https://github.com/puppetlabs/modulesync_configs ) :)

These would be overwritten anyway on the next modulesync.

  • .gitignore
  • .rubocop.yml
  • .travis.yml
  • Gemfile
  • Rakefile
  • appveyor.yml
  • spec/spec_helper.rb

@bmjen
Copy link
Contributor Author

bmjen commented Dec 15, 2017

Unfortunately, we don't have the capability of ignoring files at the moment. That's definitely something we'll have to add to the convert feature.

@bmjen
Copy link
Contributor Author

bmjen commented Dec 15, 2017

@willmeek also, with this module being converted now, we may think about removing it from the modulesync managed modules, and making PRs to pdk-templates when changes are needed/wanted.

@HelenCampbell
Copy link
Contributor

PR to Remove motd from modulesync - puppetlabs/modulesync_configs#172

@willmeek willmeek merged commit 4c155b8 into puppetlabs:master Dec 15, 2017
@bmjen bmjen deleted the pdk-convert branch December 15, 2017 15:43
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.

4 participants