-
Notifications
You must be signed in to change notification settings - Fork 192
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
test: Use ora
to make lint output less verbose
#362
Conversation
@@ -1,6 +1,8 @@ | |||
language: node_js | |||
node_js: stable | |||
sudo: false | |||
before_install: "curl -L https://unpkg.com/@pnpm/self-installer | node" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the reason to use pnpm
? I think, it makes things more complicated where unnecessary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pnpm is the third largest node package manager, which uses symlinks and isolated trees to prevent code from accessing packages which they haven’t declared in their dependencies.
See also:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linking the related conversation in mdn/mdn here: mdn/mdn#69
This copies the changes from mdn/browser-compat-data#2528.
(Preferably, we should keep common functions like schema and style validation for MDN repos in a scoped
@mdn/*
package)