You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Oct 13, 2024. It is now read-only.
For making Windows build, I wanted both portable and installer version in the same build step.
I didn't want to run the build step twice, once for each target because each time the build step executes it takes 6 minutes just to set everything up. (I'm not sure if I run the build step again in the same workflow it would re-use existing resources, thus won't take the 6 minutes to setup. I assume it would still do everything from scratch. Please correct me if I'm wrong.)
After testing (for which I wish the GitHub action validated early what was provided in the args option and failed already before installing all the dependencies and setting up - topic for another day), I found that it is possible simply by listing the targets in args separated by spaces. (You shouldn't separate by comma, it wouldn't work.)
For making Windows build, I wanted both portable and installer version in the same build step.
I didn't want to run the build step twice, once for each target because each time the build step executes it takes 6 minutes just to set everything up. (I'm not sure if I run the build step again in the same workflow it would re-use existing resources, thus won't take the 6 minutes to setup. I assume it would still do everything from scratch. Please correct me if I'm wrong.)
After testing (for which I wish the GitHub action validated early what was provided in the
args
option and failed already before installing all the dependencies and setting up - topic for another day), I found that it is possible simply by listing the targets inargs
separated by spaces. (You shouldn't separate by comma, it wouldn't work.)I find that the documentation in both the Readme as well as the website doesn't mention this fact. Hope the documentation can be enhanced. Thank you 🙂
The text was updated successfully, but these errors were encountered: