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

Spelling #315

Merged
merged 7 commits into from
May 27, 2024
Merged

Spelling #315

merged 7 commits into from
May 27, 2024

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Sep 8, 2023

Fixes misspellings identified by the check-spelling action.

The misspellings have been reported at https://github.com/jsoref/pyhocon/actions/runs/6124974527/attempts/1#summary-16626031396

The action reports that the changes in this PR would make it happy: https://github.com/jsoref/pyhocon/actions/runs/6124974600/attempts/1#summary-16626031688

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
* Fixed dictionary substititution merge. PR[#52]
* Fixed dictionary substitution merge. PR[#52]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

some projects don't like changing changelogs. Happy to drop this or anything else.

@@ -388,7 +388,7 @@ assert config == d
- Scott Johnson ([@scottj97](https://github.com/scottj97))
- Yifei Tao ([@yifeitao](https://github.com/yifeitao))
- Kevin Fong ([@KevinMFong](https://github.com/KevinMFong))
- Erik Cederstrand [@ecederstrand](https://github.com/ecederstrand))
- Erik Cederstrand ([@ecederstrand](https://github.com/ecederstrand))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This isn't precisely a spelling error, it's a stylistic error that turned up as I was tuning check-spelling to ignore the items in this list.

@darthbear darthbear merged commit 78c174f into chimpler:master May 27, 2024
1 check passed
@jsoref jsoref deleted the spelling branch May 27, 2024 11:03
@darthbear
Copy link
Member

Thanks for your contribution @jsoref! Released new version 0.3.61

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.

None yet

2 participants