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

Update .travis.yml #23

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update .travis.yml #23

wants to merge 2 commits into from

Conversation

arniu
Copy link

@arniu arniu commented Mar 26, 2020

Drop support of old node.js

.travis.yml Outdated
- '4'
- node
- lts/12
- lts/10

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Travis didn't run for this PR, maybe just ?

- 12
- 10

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's fine.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://travis-ci.org/github/kevva/to-ico/builds/667045841

$ nvm install lts/12
Version '' (with LTS filter '12') not found - try `nvm ls-remote --lts=12` to browse available versions.
Failed to install lts/12. Remote repository may not be reachable.
Using locally available version lts/12, if applicable.
$ nvm use lts/12
N/A: version "lts/12 -> N/A" is not yet installed.
ou need to run "nvm install lts/12" to install it before using it.
Unable to use lts/12
The command "false" failed and exited with 1 during .
Your build has been stopped.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, my bad.

According to [travis doc](https://docs.travis-ci.com/user/languages/javascript-with-nodejs/#specifying-nodejs-versions), we can specify Node.js versions like this:

- `node` latest stable Node.js release
- `lts/*` latest LTS Node.js release
- `14` latest 14.x release
@arniu
Copy link
Author

arniu commented May 3, 2020

How about just the latest stable and latest LTS Node.js release?

@thisconnect
Copy link

How about just the latest stable and latest LTS Node.js release?

@kevva ?

@sentience
Copy link

sentience commented Aug 20, 2020

Anything I can do to help with this, so that #19 can merge? My goal is to resolve the npm audit warning for url-regex, which is a dependency of resize-img 1.x.

Looks like we just need @kevva to make a call on this.

@bj00rn
Copy link

bj00rn commented Jan 21, 2021

@An6r @kevva can we get this merged so we can move on to #19? 🌹 🌹 🌹

jantimon/favicons-webpack-plugin#226

@chrislujan
Copy link

@kevva any chance this can get a release? #19 is dependent on this and would please the community

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

Successfully merging this pull request may close these issues.

None yet

7 participants