Skip to content

Conversation

@bartekpacia
Copy link
Collaborator

@bartekpacia bartekpacia commented Dec 3, 2024

Quick backport of #332

kudos to @penkzhou

@bartekpacia bartekpacia merged commit 8535a80 into main Dec 3, 2024
@bartekpacia bartekpacia changed the title Feature/yq on windows Automatically install yq on Windows runners Dec 3, 2024
@bartekpacia bartekpacia deleted the feature/yq_on_windows branch December 3, 2024 19:18
@solsticedhiver
Copy link

Well, this pulls another dependency, that is bash. Let me explain.

I am trying to test a github action using act. To do so, I am running act on Windows with act -P windows-latest=-self-hosted avoiding the use of docker.

I rewrote some steps to use powershell instead of bash, because I don't have bash installed in my Windows VM. but now the step using subosito/flutter/action fails with Make yq tool available on Windows runners.

I thought I could avoid that by installing yq. But that does not help.
So before instaling yq, check it is not already installed, and/or avoid using bash, when powershell is possible...

@bartekpacia
Copy link
Collaborator Author

bartekpacia commented Apr 14, 2025

Hi @solsticedhiver, sorry about this.

I'll be glad to review a PR fixing this!

@solsticedhiver
Copy link

oh wait. I am seeing that you are using bash for the rest too. That would be too complicated.

Never mind.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants