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

I know the correction for sure #781

Closed
szepeviktor opened this issue Jul 29, 2023 · 4 comments · Fixed by #884
Closed

I know the correction for sure #781

szepeviktor opened this issue Jul 29, 2023 · 4 comments · Fixed by #884

Comments

@szepeviktor
Copy link
Contributor

`Trailling` -> `Trailing`, `Trialling`, `Trilling`

"Trailling" is for sure the typo of "Trailing".
How can I configure typos to fix it right away?

@epage
Copy link
Collaborator

epage commented Jul 29, 2023

extend-words in the config can extend the available corrections used

@szepeviktor
Copy link
Contributor Author

szepeviktor commented Jul 29, 2023

You are right!!

After adding

[default.extend-words]
"trailling" = "trailing"

The fix is reduced to a one way

error: `Trailling` should be `trailing`

Thank you.

@szepeviktor
Copy link
Contributor Author

Oh no!
"Trailling" lost its capital, got turned into "trailing".

- public function noErrorOnTraillingSemicolonAndWhitespace(Connection $connection)
+ public function noErrorOntrailingSemicolonAndWhitespace(Connection $connection)

@szepeviktor szepeviktor reopened this Jul 29, 2023
@szepeviktor
Copy link
Contributor Author

szepeviktor commented Jul 29, 2023

If there are both uppercase and lowercase versions of this typo, what version Should I add?

"trailling" = "trailing"
#
"Trailling" = "Trailing"

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 a pull request may close this issue.

2 participants