Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #2 from jarretlavallee/pdk_1.6.0
(maint) Update to PDK 1.6.0 with some basic checks
- Loading branch information
Showing
12 changed files
with
351 additions
and
120 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
.git/ | ||
.*.sw[op] | ||
.metadata | ||
.yardoc | ||
.yardwarns | ||
*.iml | ||
/.bundle/ | ||
/.idea/ | ||
/.vagrant/ | ||
/coverage/ | ||
/bin/ | ||
/doc/ | ||
/Gemfile.local | ||
/Gemfile.lock | ||
/junit/ | ||
/log/ | ||
/pkg/ | ||
/spec/fixtures/manifests/ | ||
/spec/fixtures/modules/ | ||
/tmp/ | ||
/vendor/ | ||
/convert_report.txt | ||
/update_report.txt | ||
.DS_Store |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
.git/ | ||
.*.sw[op] | ||
.metadata | ||
.yardoc | ||
.yardwarns | ||
*.iml | ||
/.bundle/ | ||
/.idea/ | ||
/.vagrant/ | ||
/coverage/ | ||
/bin/ | ||
/doc/ | ||
/Gemfile.local | ||
/Gemfile.lock | ||
/junit/ | ||
/log/ | ||
/pkg/ | ||
/spec/fixtures/manifests/ | ||
/spec/fixtures/modules/ | ||
/tmp/ | ||
/vendor/ | ||
/convert_report.txt | ||
/update_report.txt | ||
.DS_Store |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
--color | ||
--format documentation |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,118 @@ | ||
--- | ||
require: rubocop-rspec | ||
AllCops: | ||
DisplayCopNames: true | ||
TargetRubyVersion: '2.1' | ||
Include: | ||
- "./**/*.rb" | ||
Exclude: | ||
- bin/* | ||
- ".vendor/**/*" | ||
- "**/Gemfile" | ||
- "**/Rakefile" | ||
- pkg/**/* | ||
- spec/fixtures/**/* | ||
- vendor/**/* | ||
- "**/Puppetfile" | ||
- "**/Vagrantfile" | ||
- "**/Guardfile" | ||
Metrics/LineLength: | ||
Description: People have wide screens, use them. | ||
Max: 200 | ||
RSpec/BeforeAfterAll: | ||
Description: Beware of using after(:all) as it may cause state to leak between tests. | ||
A necessary evil in acceptance testing. | ||
Exclude: | ||
- spec/acceptance/**/*.rb | ||
RSpec/HookArgument: | ||
Description: Prefer explicit :each argument, matching existing module's style | ||
EnforcedStyle: each | ||
Style/BlockDelimiters: | ||
Description: Prefer braces for chaining. Mostly an aesthetical choice. Better to | ||
be consistent then. | ||
EnforcedStyle: braces_for_chaining | ||
Style/ClassAndModuleChildren: | ||
Description: Compact style reduces the required amount of indentation. | ||
EnforcedStyle: compact | ||
Style/EmptyElse: | ||
Description: Enforce against empty else clauses, but allow `nil` for clarity. | ||
EnforcedStyle: empty | ||
Style/FormatString: | ||
Description: Following the main puppet project's style, prefer the % format format. | ||
EnforcedStyle: percent | ||
Style/FormatStringToken: | ||
Description: Following the main puppet project's style, prefer the simpler template | ||
tokens over annotated ones. | ||
EnforcedStyle: template | ||
Style/Lambda: | ||
Description: Prefer the keyword for easier discoverability. | ||
EnforcedStyle: literal | ||
Style/RegexpLiteral: | ||
Description: Community preference. See https://github.com/voxpupuli/modulesync_config/issues/168 | ||
EnforcedStyle: percent_r | ||
Style/TernaryParentheses: | ||
Description: Checks for use of parentheses around ternary conditions. Enforce parentheses | ||
on complex expressions for better readability, but seriously consider breaking | ||
it up. | ||
EnforcedStyle: require_parentheses_when_complex | ||
Style/TrailingCommaInArguments: | ||
Description: Prefer always trailing comma on multiline argument lists. This makes | ||
diffs, and re-ordering nicer. | ||
EnforcedStyleForMultiline: comma | ||
Style/TrailingCommaInLiteral: | ||
Description: Prefer always trailing comma on multiline literals. This makes diffs, | ||
and re-ordering nicer. | ||
EnforcedStyleForMultiline: comma | ||
Style/SymbolArray: | ||
Description: Using percent style obscures symbolic intent of array's contents. | ||
EnforcedStyle: brackets | ||
RSpec/MessageSpies: | ||
EnforcedStyle: receive | ||
Style/Documentation: | ||
Exclude: | ||
- lib/puppet/parser/functions/**/* | ||
- spec/**/* | ||
Style/WordArray: | ||
EnforcedStyle: brackets | ||
Style/CollectionMethods: | ||
Enabled: true | ||
Style/MethodCalledOnDoEndBlock: | ||
Enabled: true | ||
Style/StringMethods: | ||
Enabled: true | ||
Layout/EndOfLine: | ||
Enabled: false | ||
Layout/IndentHeredoc: | ||
Enabled: false | ||
Metrics/AbcSize: | ||
Enabled: false | ||
Metrics/BlockLength: | ||
Enabled: false | ||
Metrics/ClassLength: | ||
Enabled: false | ||
Metrics/CyclomaticComplexity: | ||
Enabled: false | ||
Metrics/MethodLength: | ||
Enabled: false | ||
Metrics/ModuleLength: | ||
Enabled: false | ||
Metrics/ParameterLists: | ||
Enabled: false | ||
Metrics/PerceivedComplexity: | ||
Enabled: false | ||
RSpec/DescribeClass: | ||
Enabled: false | ||
RSpec/ExampleLength: | ||
Enabled: false | ||
RSpec/MessageExpectation: | ||
Enabled: false | ||
RSpec/MultipleExpectations: | ||
Enabled: false | ||
RSpec/NestedGroups: | ||
Enabled: false | ||
Style/AsciiComments: | ||
Enabled: false | ||
Style/IfUnlessModifier: | ||
Enabled: false | ||
Style/SymbolProc: | ||
Enabled: false |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
.gitlab-ci.yml: | ||
delete: true | ||
appveyor.yml: | ||
delete: true | ||
|
||
Rakefile: | ||
extras: | ||
- | | ||
desc 'Syntax check shellscripts' | ||
task :shellcheck do | ||
Dir['{tasks,files}/**/*.sh'].each do |shell_file| | ||
sh "shellcheck #{shell_file}" do |ok, res| | ||
exit res.exitstatus unless ok | ||
end | ||
end | ||
end | ||
.travis.yml: | ||
branches: | ||
- /.*/ | ||
deploy: false | ||
includes: | ||
- | ||
env: CHECK="syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop shellcheck" | ||
remove_includes: | ||
- | ||
env: CHECK="syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop" | ||
- | ||
env: CHECK=parallel_spec | ||
- | ||
env: PUPPET_GEM_VERSION="~> 4.0" CHECK=parallel_spec | ||
rvm: 2.1.9 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
--- | ||
sudo: false | ||
dist: trusty | ||
language: ruby | ||
cache: bundler | ||
before_install: | ||
- bundle -v | ||
- rm -f Gemfile.lock | ||
- gem update --system | ||
- gem --version | ||
- bundle -v | ||
script: | ||
- 'bundle exec rake $CHECK' | ||
bundler_args: --without system_tests | ||
rvm: | ||
- 2.4.1 | ||
env: | ||
global: | ||
- BEAKER_PUPPET_COLLECTION=puppet5 PUPPET_GEM_VERSION="~> 5.0" | ||
matrix: | ||
fast_finish: true | ||
include: | ||
- | ||
env: CHECK="syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop shellcheck" | ||
branches: | ||
only: | ||
- master | ||
- /^v\d/ | ||
- /.*/ | ||
notifications: | ||
email: false |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
--markup markdown |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.