Skip to content

Commit

Permalink
Test NodeJS 20 on all ELs
Browse files Browse the repository at this point in the history
  • Loading branch information
evgeni committed Jan 3, 2024
1 parent f570e7c commit 8e0c806
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions spec/acceptance/class_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -106,8 +106,7 @@ class { 'nodejs':
end
end

context 'RedHat with repo_class => nodejs::repo::dnfmodule', if: fact('os.family') == 'RedHat' && %w[8 9].include?(fact('os.release.major')), skip: ((nodejs_version == '20' && fact('os.name') != 'CentOS') || (nodejs_version == '16' && fact('os.release.major') == '9') ? 'NodeJS 20 is not yet in a released EL, NodeJS 16 is not available on EL9' : nil) do
# Node 20 is only available in Stream yet, not in a released EL
context 'RedHat with repo_class => nodejs::repo::dnfmodule', if: fact('os.family') == 'RedHat' && %w[8 9].include?(fact('os.release.major')), skip: ((nodejs_version == '16' && fact('os.release.major') == '9') ? 'NodeJS 16 is not available on EL9' : nil) do

Check failure on line 109 in spec/acceptance/class_spec.rb

View workflow job for this annotation

GitHub Actions / Puppet / Static validations

Style/TernaryParentheses: Omit parentheses for ternary conditions.
# Node 16 is not available on EL9

include_examples 'cleanup'
Expand Down

0 comments on commit 8e0c806

Please sign in to comment.