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

Typos a/an #13812

Merged
merged 1 commit into from
Dec 1, 2022
Merged

Typos a/an #13812

merged 1 commit into from
Dec 1, 2022

Conversation

dilyanpalauzov
Copy link
Contributor

No description provided.

@fwolter
Copy link
Member

fwolter commented Dec 1, 2022

Thank you for this PR! Catching also things like "a {@link AsynchronousSocketChannel}" looks like a bit more sophisticated script. I guess you didn't do that by hand...?

@dilyanpalauzov
Copy link
Contributor Author

I found the places with git grep and then changed the files by hand.

Copy link
Member

@fwolter fwolter left a comment

Choose a reason for hiding this comment

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

Thanks, good work!

@fwolter
Copy link
Member

fwolter commented Dec 1, 2022

There's a formatting error due to a line, which became one character longer. Can you run mvn spotless:apply?

@fwolter fwolter merged commit 78534e8 into openhab:main Dec 1, 2022
@fwolter fwolter added this to the 3.4 milestone Dec 1, 2022
@dilyanpalauzov dilyanpalauzov deleted the typos_a_an branch December 1, 2022 14:58
andrasU pushed a commit to andrasU/openhab-addons that referenced this pull request Dec 24, 2022
Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
borazslo pushed a commit to borazslo/openhab-mideaac-addon that referenced this pull request Jan 8, 2023
psmedley pushed a commit to psmedley/openhab-addons that referenced this pull request Feb 23, 2023
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Feb 28, 2023
andrasU pushed a commit to andrasU/openhab-addons that referenced this pull request Jan 6, 2024
Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants