search

voxpupuli / puppet-nodejs

Puppet module to install nodejs and global npm packages
https://forge.puppet.com/puppet/nodejs
Apache License 2.0
113 stars 247 forks source link

update list of tested OSes accordingly with metadata.json #414

Closed mmoll closed 4 years ago

mmoll commented 4 years ago

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

mmoll commented 4 years ago

rebased version of GH-402

mmoll commented 4 years ago

Debian/buster is already bringing NodeJS 10, so using 8.x as default will not work, as the package is not downgraded, leading to a test fail.

Looking at https://nodejs.org/en/about/releases/ I'd even lean towards setting the default to 12.x.

juniorsysadmin commented 4 years ago

Also consider updating the repo_url_suffix section of the README

mmoll commented 4 years ago

Also consider updating the repo_url_suffix section of the README

Done.