-
Notifications
You must be signed in to change notification settings - Fork 31
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
[fix] Update git actions #127
Conversation
- Update actions in `ci.yml` - Bump node version from 12 => 16 - Switch from `yarn` to `npm` - Replace `loadimpact/har-to-k6` with `grafana/har-to-k6` in `package.json` - Update `package.json~engines.node` - Bump "unsafe" packages
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.
LGTM 🚢
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.
For some reason npm run test
fails for me locally with TypeError: process.stderr.hasColors is not a function
, but since it works in CI, it must be an issue with my machine.
LGTM! 🚀
Nice catch! It behaved the same in CI, just didnt count as "fail". |
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.
Works perfectly now! 💯
ci.yml
yarn
tonpm
loadimpact/har-to-k6
withgrafana/har-to-k6
inpackage.json
package.json~engines.node
npm publish