Skip to content
ci

(CAT-1869) - Add configurable dsc_timeout #741

Sign in to view logs

(CAT-1869) - Add configurable dsc_timeout

(CAT-1869) - Add configurable dsc_timeout #741

Triggered via pull request June 10, 2024 13:48
@jordanbreen28jordanbreen28
opened #319
cat-1869-add_configurable_timeout
Status Failure
Total duration 37s
Artifacts –

ci.yml

on: pull_request
Matrix: spec
Matrix: acceptance
Waiting for pending jobs
Fit to window
Zoom out
Zoom in

Annotations

14 errors
spec (ruby 3.2 | puppet ~> 8.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L263
Style/StringLiterals: Prefer single-quoted strings when you don't need string interpolation or special symbols.
spec (ruby 3.2 | puppet ~> 8.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L307
Layout/TrailingWhitespace: Trailing whitespace detected.
spec (ruby 3.2 | puppet ~> 8.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L309
Naming/AccessorMethodName: Do not prefix writer method names with `set_`.
spec (ruby 3.2 | puppet ~> 8.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L310
Style/GuardClause: Use a guard clause (`return unless name_hash[:dsc_timeout] && (@timeout == nil || @timeout != name_hash[:dsc_timeout])`) instead of wrapping the code inside a conditional expression.
spec (ruby 3.2 | puppet ~> 8.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L310
Style/IfUnlessModifier: Favor modifier `if` usage when having a single-line body. Another good alternative is the usage of control flow `&&`/`||`.
spec (ruby 3.2 | puppet ~> 8.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L310
Style/NilComparison: Prefer the use of the `nil?` predicate.
spec (ruby 3.2 | puppet ~> 8.0) / spec
Process completed with exit code 1.
spec (ruby 2.7 | puppet ~> 7.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L263
Style/StringLiterals: Prefer single-quoted strings when you don't need string interpolation or special symbols.
spec (ruby 2.7 | puppet ~> 7.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L307
Layout/TrailingWhitespace: Trailing whitespace detected.
spec (ruby 2.7 | puppet ~> 7.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L309
Naming/AccessorMethodName: Do not prefix writer method names with `set_`.
spec (ruby 2.7 | puppet ~> 7.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L310
Style/GuardClause: Use a guard clause (`return unless name_hash[:dsc_timeout] && (@timeout == nil || @timeout != name_hash[:dsc_timeout])`) instead of wrapping the code inside a conditional expression.
spec (ruby 2.7 | puppet ~> 7.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L310
Style/IfUnlessModifier: Favor modifier `if` usage when having a single-line body. Another good alternative is the usage of control flow `&&`/`||`.
spec (ruby 2.7 | puppet ~> 7.0) / spec: lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb#L310
Style/NilComparison: Prefer the use of the `nil?` predicate.
spec (ruby 2.7 | puppet ~> 7.0) / spec
Process completed with exit code 1.