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

Add puppet-lint github action? #671

Closed
jcpunk opened this issue Sep 14, 2020 · 3 comments
Closed

Add puppet-lint github action? #671

jcpunk opened this issue Sep 14, 2020 · 3 comments

Comments

@jcpunk
Copy link
Contributor

jcpunk commented Sep 14, 2020

Would it be worth adding the puppet-lint github action to the default workflow for voxpupuli modules?

https://github.com/marketplace/actions/puppet-lint-action

@ekohl
Copy link
Member

ekohl commented Sep 14, 2020

No, it doesn't. It's part of the Travis config and already runs. That action in particular uses a pre-built container and only runs puppet-lint in it. That means we don't get the plugins that we rely on.

@jcpunk
Copy link
Contributor Author

jcpunk commented Sep 15, 2020

Interesting.

I've been opening PRs that resolve puppet-lint issues I've randomly discovered in various voxpupuli modules the last few days. Is Travis only checking changed lines?

@ekohl
Copy link
Member

ekohl commented Sep 15, 2020

It shouldn't, but not all modules are in sync with our current best practices (first check if there's a modulesync PR open). Another thing is that if you have some plugins in ~/.gem, it doesn't mean we are using them in our modules. Those are some things you may see.

@ekohl ekohl closed this as completed Nov 6, 2020
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

No branches or pull requests

2 participants