Skip to content

fix(workspaces): add and arrange missing env vars #643

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

Merged
merged 7 commits into from
Oct 19, 2021

Conversation

greyscaled
Copy link
Contributor

Need to fill in the details of a few of them. Arranged them to match the output given by

env | grep CODER_

env-vars

@greyscaled greyscaled self-assigned this Oct 1, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Oct 1, 2021

✨ Coder.com for PR #643 deployed! It will be updated on every commit.

@greyscaled greyscaled force-pushed the vapurrmaid/missing-workspace-env-vars branch from 4c46a98 to 6f7cf8d Compare October 18, 2021 19:25
@greyscaled greyscaled requested review from khorne3 and jawnsy October 18, 2021 19:28
<tr>
<td><code>CODER_ASSETS_ROOT</code></td>
<td>The directory where coder adds coder-specific assets during
workspace creation, such as the coder-cli binary</td>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
workspace creation, such as the coder-cli binary</td>
workspace creation, such as the `coder-cli` binary</td>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think we'll have to use <code> wrappers here since this part is within a <td>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

pushed in dc86f04 - will need to check preview

greyscaled and others added 2 commits October 18, 2021 16:10
Co-authored-by: Katie Horne <katie@coder.com>
@greyscaled greyscaled marked this pull request as ready for review October 18, 2021 20:25
@greyscaled greyscaled requested a review from khorne3 October 18, 2021 20:26
@greyscaled
Copy link
Contributor Author

Thanks for the added commits @khorne3 🎉 !!

@greyscaled greyscaled merged commit f7e88bd into main Oct 19, 2021
@greyscaled greyscaled deleted the vapurrmaid/missing-workspace-env-vars branch October 19, 2021 15:55
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