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

Fix helm chart for extraConfigmapMounts #4497

Merged
merged 1 commit into from
Nov 12, 2021
Merged

Conversation

jlandowner
Copy link
Contributor

Fixes #4496

@codecov
Copy link

codecov bot commented Nov 11, 2021

Codecov Report

Merging #4497 (ce65c0a) into main (d052cc2) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4497   +/-   ##
=======================================
  Coverage   66.46%   66.46%           
=======================================
  Files          30       30           
  Lines        1625     1625           
  Branches      330      330           
=======================================
  Hits         1080     1080           
  Misses        463      463           
  Partials       82       82           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d052cc2...ce65c0a. Read the comment docs.

@jsjoeio jsjoeio self-assigned this Nov 12, 2021
Copy link
Contributor

@jsjoeio jsjoeio left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you so much!

@jsjoeio jsjoeio merged commit cdf0deb into coder:main Nov 12, 2021
@jlandowner
Copy link
Contributor Author

Thanks!

GirlBossRush pushed a commit that referenced this pull request Nov 15, 2021
GirlBossRush pushed a commit that referenced this pull request Nov 15, 2021
* Refactor vscode router to load async.

* Bump vscode.

* fix volumes (#4497)

* Fix : recreate the termux guide to adapt the recent changes (#4472)

* Fix : recreate the termux guide to adapt the recent changes

Termux nodejs-lts changed from v14 to v16 and there are many issues people are facing such as with argon2. Hence I recommend changing it to this install process which is comparably better and has one less issue :^)

I've also added some extra things such as installing GO and Python, idk about the TOC tree but this is pretty much it.

* yarn-fmt and minor typos

#4472 (comment)

* Fix : replace unnecessary steps to be linked to a guide

* Change from private gist to a section in Extra

* Remove reference to non-existent step

* ready to merge!

Co-authored-by: Joe Previte <[email protected]>

Co-authored-by: Jinu <[email protected]>
Co-authored-by: Han Seung Min - 한승민 <[email protected]>
Co-authored-by: Joe Previte <[email protected]>
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.

Invalid helm chart when extraConfigmapMounts specified
2 participants