-
Notifications
You must be signed in to change notification settings - Fork 5
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
revert product-os/flowzone/pull/953 #955
Conversation
I prefer this approach rather than hardcoding runner tags. |
How is this approach fixing the underlying condition that the setup-python action will only currently succeed on X64/Linux? I must be missing something, perhaps you can add you commit directly to this branch? We have seven environment repositories that are affected by this. We could technically change them all to only run on X64/Linux, but I think a one line change here is better than having to change x7, especially given that it will be temporary. |
HOW temporary? Do we know if/when this will be fixed upstream? I see the benefit of fixing it in Flowzone as it's less work, but then we have to make assumptions about what the calling repository has access to (self-hosted vs GitHub hosted) AND we need to remember and come back later to remove all the places in Flowzone where we are ignoring the Would this be a suitable compromise? |
Ugh.. |
change-type: patch