-
-
Notifications
You must be signed in to change notification settings - Fork 217
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
chore: update pkg.pr.new
gh action
#230
Conversation
/pkg-pr-new |
package.json
Outdated
@@ -6,13 +6,13 @@ | |||
"description": "Use the right package manager", | |||
"author": "Anthony Fu <anthonyfu117@hotmail.com>", | |||
"license": "MIT", | |||
"homepage": "https://github.com/antfu-collective/ni#readme", | |||
"homepage": "https://github.com/antfu/ni#readme", |
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.
Why do we change back on this?
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.
Doing some test about using --compact
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.
I Will revert this tmr, or feel free to revert changes yourself.
I asked Moha to help us with the behaviour:
- release on every commit on main (?)
- release on any pr using
/pgk-pr-new
or some other comment (like vite ecosytem-ci)
Release on every PR commit and main branch (excluding tags)
commit:
|
We can also remove the dev dependency, removing it... dont merge yet |
Description
This PR adds
pkg.pr.new
as dev dependency, we need to registerni
here: https://github.com/apps/pkg-pr-new (I don't have admin privileges onni
repo)Linked Issues
Additional context