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 typo in docs components.md #1639

Merged
merged 2 commits into from
Sep 10, 2024
Merged

Fix typo in docs components.md #1639

merged 2 commits into from
Sep 10, 2024

Conversation

fridgepoet
Copy link
Member

@fridgepoet fridgepoet commented Sep 10, 2024

PR Description

If I understood correctly, the component is "local.file" and its label is "api_key" so the comment seemed to have a typo. Let me know if I have misunderstood!

Which issue(s) this PR fixes

Notes to the Reviewer

PR Checklist

  • CHANGELOG.md updated
  • Documentation added
  • Tests updated
  • Config converters updated

If I understood correctly, the component is "local.file" and its name (?) is "api_key" so the comment seemed to have a typo.
Copy link
Contributor

@clayton-cornell clayton-cornell left a comment

Choose a reason for hiding this comment

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

Good catch @fridgepoet Looks like a simple copy-paste error crept into the content. Than happens and they can be hard to spot some days... so thank you for submitting the PR.

I committed your alternate suggestion as I agree, that's more correct here.

@clayton-cornell clayton-cornell enabled auto-merge (squash) September 10, 2024 17:06
@clayton-cornell clayton-cornell merged commit 44282cc into main Sep 10, 2024
23 checks passed
@clayton-cornell clayton-cornell deleted the fridgepoet-patch-1 branch September 10, 2024 17:10
github-actions bot pushed a commit that referenced this pull request Sep 10, 2024
Co-authored-by: Shirley <[email protected]>
Co-authored-by: Clayton Cornell <[email protected]>
(cherry picked from commit 44282cc)
github-actions bot pushed a commit that referenced this pull request Sep 10, 2024
Co-authored-by: Shirley <[email protected]>
Co-authored-by: Clayton Cornell <[email protected]>
(cherry picked from commit 44282cc)
github-actions bot pushed a commit that referenced this pull request Sep 10, 2024
Co-authored-by: Shirley <[email protected]>
Co-authored-by: Clayton Cornell <[email protected]>
(cherry picked from commit 44282cc)
github-actions bot pushed a commit that referenced this pull request Sep 10, 2024
Co-authored-by: Shirley <[email protected]>
Co-authored-by: Clayton Cornell <[email protected]>
(cherry picked from commit 44282cc)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants