Skip to content

fix NOTE.txt #4515

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 1 commit into from
Nov 16, 2021
Merged

fix NOTE.txt #4515

merged 1 commit into from
Nov 16, 2021

Conversation

jlandowner
Copy link
Contributor

Fixes #4504

Currently fetching Pod, but I think it is better to use Service directly.

@codecov
Copy link

codecov bot commented Nov 16, 2021

Codecov Report

Merging #4515 (982bbcd) into main (0a072f7) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4515   +/-   ##
=======================================
  Coverage   66.03%   66.03%           
=======================================
  Files          30       30           
  Lines        1640     1640           
  Branches      330      330           
=======================================
  Hits         1083     1083           
  Misses        474      474           
  Partials       83       83           

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 0a072f7...982bbcd. Read the comment docs.

@jsjoeio jsjoeio self-assigned this Nov 16, 2021
@jsjoeio jsjoeio added the bug Something isn't working label Nov 16, 2021
@jsjoeio
Copy link
Contributor

jsjoeio commented Nov 16, 2021

Hey @Matthew-Beckett - do you mind helping me review this PR? I think it looks good but wanted to get your take since you know this part of the codebase better than me!

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.

Thanks for doing this 🎉

Copy link
Contributor

@Matthew-Beckett Matthew-Beckett left a comment

Choose a reason for hiding this comment

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

Changes don't affect chart function, just "cosmetic".

@Matthew-Beckett
Copy link
Contributor

Thanks for your contribution @jlandowner this is definitely better!

❤️

@jsjoeio jsjoeio merged commit ccb9d94 into coder:main Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Helm chart] Typo in Helm chart notes
3 participants