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.

Sign up for GitHub

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

Jump to bottom

Document missing bundled conda for self hosted runners #187

Merged
merged 3 commits into from May 26, 2022
Merged

Document missing bundled conda for self hosted runners #187

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0fbf43a
Document missing bundled conda for self hosted runners
jaimergp Aug 3, 2021
76b481a
mention miniforge too
jaimergp Aug 10, 2021
8ca9e40
Fix typo
jaimergp Aug 20, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 5 additions & 1 deletion src/setup.ts
View file
Open in desktop
Expand Up @@ -48,7 +48,11 @@ async function setupMiniconda(inputs: types.IActionInputs): Promise<void> {
}

if (!fs.existsSync(basePath)) {
throw Error(`No installed conda 'base' enviroment found at ${basePath}`);
throw Error(`No installed conda 'base' enviroment found at ${basePath}!` +
jaimergp marked this conversation as resolved.
Show resolved Hide resolved
'If you are using this action in a self-hosted runner that already provides ' +
'its own Miniconda installation, please specify its location with a `CONDA` ' +
'environment variable. If you want us to download and install Miniconda for ' +
'you, use `miniconda-version: "latest"` in the parameters for this action.');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might also want to rep miniforge-version...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jaimergp ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops missed this, pushing now.

}

await core.group("Setup environment variables...", () =>
Expand Down