ci: add node18 support and drop node17 ci

less / less.js

Less. The dynamic stylesheet language.

http://lesscss.org

Apache License 2.0

17k stars 3.41k forks source link

Closed iChenLei closed 2 years ago

iChenLei commented 2 years ago

What: https://nodejs.org/en/blog/announcements/v18-release-announce/

Why: Follow nodejs LTS !

How: chang ci config

Checklist:

iChenLei commented 2 years ago

BTW, why are we running so many test environments now? 🤔 Is there that much variance across OS's?

@matthew-dean Our current test matrix, only 7 tests, I think across OS test is required.

	ubuntu	macOS	windows
node	v10 v12 v14 v16 v18	v16(Latest LTS)	v16(Latest LTS)

matthew-dean commented 2 years ago

Hmm, I guess there's no real harm of testing more, if it's free. 🤔