Skip to content
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

Download and unzip to temp folder and not $HOME #132

Merged
merged 1 commit into from Feb 4, 2022

Conversation

kuhnroyal
Copy link
Contributor

Using the $HOME folder can be problematic in some constellations on private runners.
$RUNNER_TEMP is the official environment variable for a runner specific temp folder.

@subosito subosito merged commit 77740fc into subosito:main Feb 4, 2022
@subosito
Copy link
Owner

subosito commented Feb 4, 2022

Hi @kuhnroyal, it doesn't work on Windows, https://github.com/subosito/flutter-action/actions/runs/1794413874. Could you submit a PR to fix that? Thanks.

@kuhnroyal
Copy link
Contributor Author

Can you setup the pipeline to run for a PR? :)
I'll see what I can do.

@subosito
Copy link
Owner

subosito commented Feb 4, 2022

Sure, I have added that in b5a1c34.

@kuhnroyal kuhnroyal deleted the temp-folder branch February 4, 2022 11:23
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.

None yet

2 participants