-
Notifications
You must be signed in to change notification settings - Fork 13
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
GitHub Action #4
Comments
I just added a quick run to my existing checks for now. https://github.com/react-hook-form/react-hook-form/pull/3236/files#diff-5ca9504d40891cc08c8fff3c5d9be551bb2a39bd09803540bff22c36e7e7b8ea |
Yes you could to the same workflow as above, just exchange |
Here is a first version: could someone give it a try? https://github.com/gr2m/skypack-quality-score-action I gotta run now, but will check back later tonight |
Works as expected: @FredKSchott I would suggest to create a separate repository for the action: https://github.com/skypackjs/package-check-action. Once we rename the repository, I'd publish it to the marketplace so folks can find it. Once that's done, we can discuss adding more features to it, like a better report by creating a check run instead of relying on the binary status of the GitHub Action run |
Awesome! Repo created, just gave you access: https://github.com/skypackjs/package-check-action |
I'm working on creating an official GitHub Action to integrate Skypack's package-score into your CI.
Update
An action exists now:
https://github.com/gr2m/skypack-quality-score-action#readme
But we will likely move the repository to the @skypackjs org, see #4 (comment)
The plan™️
Sample workflow file
In the first version the Action would just succeed / fail similar to what the binary does. In future we could create a separate check run with a human-friendy report
The text was updated successfully, but these errors were encountered: