-
Notifications
You must be signed in to change notification settings - Fork 14.3k
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
Rewrite our Github Action workflows to Python #13182
Comments
cc: @kaxil @ashb @turbaszek @mik-laj and @TobKed - let's do it! |
🎉 Yes let's do it |
@potiuk I will like to help and contribute. |
@potiuk is it sill something considered to be done? |
This is in fact in progress. We have an intern from the Outreachy project that will take over this. |
Oh yeah. I will update the issue soon with an inventory of what is needed and we will work from here. Similarly #12282 with rewriting breeze. Those will have some overlap so the two interns will be working together. More on that soon! |
Yeah. Finally the time had come to start work on the rewrite: First, founding PR is here: #19867 - those who are interested will find there some ADR (Architecture Decision Records) which we will keep throughout the whole project to capture all important decisions we've made and once we merge it, more stuff will come. |
I believe this is now completed :) ? |
Oh. Vrery much so :). |
Yesterday (18th of December) GitHub announced a 3rd-party Python GHApi: https://github.blog/2020-12-18-learn-about-ghapi-a-new-third-party-python-client-for-the-github-api/
One of the features of the API is the ability to write GitHub Actions Workflows (including custom actions) entirely in Python:
https://github.blog/2020-12-18-learn-about-ghapi-a-new-third-party-python-client-for-the-github-api/#github-actions
This is super cool. Knowing the difficulties of combining YAML files, bash scripts and TypeScripts Actions) together, this is a so much better and easier way of writing the workflows,
We should rewrite all our workflows using this new API !
The text was updated successfully, but these errors were encountered: