Skip to content

Conversation

alexrockhill
Copy link
Contributor

From testing, this was actually using the system-wide python install so probably should set up its own for a better config.

@YuriiMotov
Copy link
Member

@alexrockhill, thanks for interest in this project!

The re-actors/alls-green cation only has standard dependencies and doesn't have any specific requirements on Python version.
Also, current implementation is inline with the official example.

Please, explain why you think this is important.
Installing Python will increase execution time. Do we need this?

@alexrockhill
Copy link
Contributor Author

I ran this on a local runner without a system wide python installation and it failed because all-greens calls a plain python command within it.

@github-actions github-actions bot removed the waiting label Sep 3, 2025
@YuriiMotov
Copy link
Member

I ran this on a local runner without a system wide python installation and it failed because all-greens calls a plain python command within it.

Then I would make this step conditional (if no Python 3 installed - install)

@stevleibelt
Copy link

@YuriiMotov how important is execution/build time compared to "comes with all batteries included"?

The less external and uncontrollable dependencies, the better and streamlined the result, or?

@alejsdev
Copy link
Member

alejsdev commented Sep 9, 2025

Hi @alexrockhill 👋

This job is to determine if a PR can be merged, and since we don't use local runners, we don't really need this change here, but feel free to add it to your own project. Given that, I'll pass on this one. But thanks for the effort! 🚀

@alejsdev alejsdev closed this Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants