search

m4rs-mt / ILGPU

ILGPU JIT Compiler for high-performance .Net GPU programs
http://www.ilgpu.net
Other
1.34k stars 115 forks source link

[FEATURE REQUEST]: Improve check-required workflow #1248

Open pavlovic-ivan opened 2 months ago

pavlovic-ivan commented 2 months ago

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

pavlovic-ivan commented 2 months ago

@MoFtZ @m4rs-mt how do you feel about this request?

MoFtZ commented 2 months ago

Sounds like a good idea @pavlovic-ivan 👍 Especially if it makes your life easier.

pavlovic-ivan commented 2 months ago

All right, happy to hear that. I will create a PR for you soon

pavlovic-ivan commented 2 months ago

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.

MoFtZ commented 2 months ago

There will be some setup required in the settings

This will need to be handled by @m4rs-mt 🥰

m4rs-mt commented 1 month ago

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

pavlovic-ivan commented 1 month ago

Sounds good @m4rs-mt , let's sync offline