-
Notifications
You must be signed in to change notification settings - Fork 122
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
[FEATURE REQUEST]: Improve check-required workflow #1248
Comments
Sounds like a good idea @pavlovic-ivan 👍 Especially if it makes your life easier. |
All right, happy to hear that. I will create a PR for you soon |
There will be some setup required in the settings, if that's ok with you @MoFtZ ? Basically, a Github app needs to be created, and it's app id and private key should be added as secrets to ILGPU settings. |
This will need to be handled by @m4rs-mt 🥰 |
I'm back and happy to do what is required in the next days! Will connect with you offline to get things going @pavlovic-ivan |
Sounds good @m4rs-mt , let's sync offline |
Is your feature request related to a problem? Please describe.
N/A
Describe the solution you'd like
In several other repos we maintain (like Parquetsharp, FML, YHS, Armada), we have the latest version of check-required workflow. This version is a bit more advanced than the currently existing one in ILGPU. I would like to contribute with the improvement of this workflow, so i can, later on, extract it to a common repo, where it would be centralised, and updated regularly, so everyone receives the latest version
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: