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

Linter to check spelling of file name? #2009

Closed
kcfedun-fincad opened this issue Oct 25, 2022 · 7 comments · Fixed by #2022
Closed

Linter to check spelling of file name? #2009

kcfedun-fincad opened this issue Oct 25, 2022 · 7 comments · Fixed by #2022
Labels
question Further information is requested

Comments

@kcfedun-fincad
Copy link

Hello,

Does MegaLinter support a linter that checks the spelling of file names?

@kcfedun-fincad kcfedun-fincad added the question Further information is requested label Oct 25, 2022
@Kurt-von-Laven
Copy link
Collaborator

Not by default, but the wonderful maintainer of CSpell has published a variety of workarounds: streetsidesoftware/cspell#3063. I recommend upvoting the issue and/or chiming in if those don’t meet your needs.

@nvuillam
Copy link
Member

Not by default, but the wonderful maintainer of CSpell has published a variety of workarounds: streetsidesoftware/cspell#3063. I recommend upvoting the issue and/or chiming in if those don’t meet your needs.

MegaLinter also has great maintainers... We can probably do such thing natively in MegaLinter, maybe by generating a temporary text file with file names and add it to the list of files sent to cspell ^^

@Kurt-von-Laven
Copy link
Collaborator

It’s certainly doable, but given that Jason doesn’t seem opposed to adding such a feature to CSpell, that feels like a more appropriate home for this functionality?

@nvuillam
Copy link
Member

The issue exists for 4 months, we can probably solve it in 2 hours within MegaLinter, let's allow ML to provide additional value to existing linters 😇

@Kurt-von-Laven
Copy link
Collaborator

I think the issue has been open because Jason himself has no particular need for this feature, and he raised some good questions about how to implement it correctly. I am not so sure it will be any less work to add a correct implementation to MegaLinter than directly to CSpell though?

@nvuillam
Copy link
Member

More simple for me to do it in ML, where we already have the list of files so have only one case to manage ^^

@Kurt-von-Laven
Copy link
Collaborator

I see your point.

nvuillam added a commit that referenced this issue Oct 31, 2022
Allow cspell to also analyze file names
Fixes #2009
nvuillam added a commit that referenced this issue Oct 31, 2022
* Cspell: SPELL_CSPELL_ANALYZE_FILE_NAMES

Allow cspell to also analyze file names
Fixes #2009

* [MegaLinter] Apply linters fixes

Co-authored-by: nvuillam <nvuillam@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants