Add spell file extensions parameter

[bdovaz]

Related #1997

@nvuillam questions:

1. What about: https://github.com/oxsecurity/megalinter/blob/main/.automation/build.py#L1255 ???
2. I don't know if the descriptor change is correct, I want to expose that parameter but without a default value so it would behave like now without any behaviour change.

[codecov-commenter]

Codecov Report

Merging #2001 (20261a8) into main (a47459f) will increase coverage by 0.67%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2001      +/-   ##
==========================================
+ Coverage   82.03%   82.71%   +0.67%     
==========================================
  Files         157      157              
  Lines        3384     3384              
==========================================
+ Hits         2776     2799      +23     
+ Misses        608      585      -23     

Impacted Files	Coverage Δ
megalinter/reporters/UpdatedSourcesReporter.py	89.74% <0.00%> (+2.56%)	⬆️
...alinter/tests/test_megalinter/helpers/utilstest.py	89.01% <0.00%> (+8.05%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[nvuillam]

Plz check comments :)

• update CHANGELOG.md :)

[nvuillam]

I want default behaviour to remain the current one :)
So please don't update the descriptor ^^

[bdovaz]

But if I don't update the descriptor, neither the schema nor the documentation will be updated to reflect that the SPELL_*_FILE_EXTENSIONS parameter can be set, right?

[nvuillam]

You're right :)
To do that, you must remove the condition here :)
https://github.com/bdovaz/megalinter/blob/d03b3efdcb48eefbd813dce5f5628705f6cbc664/.automation/build.py#L1255

[bdovaz]

I just realized that even removing that second condition won't work because the value of "lint_all_other_linters_files" is set to true: https://github.com/bdovaz/megalinter/blob/main/megalinter/descriptors/spell.megalinter-descriptor.yml

The condition in the script you mentioned compares it to false

[bdovaz]

Plz check comments :)

• update CHANGELOG.md :)

Updated

[bdovaz]

@nvuillam Can you help me to unlock this PR? Thank you.

There is some unresolved conversation I need your guidance on.

[nvuillam]

@bdovaz the PR seems fine to me , but i've not validated it yet because it's still in status "draft" :)
You can merge conflicts and declare it as ready for merge, and i'll manage it :)

[bdovaz]

@nvuillam done!

[nvuillam]

Thanks a lot for your contribution :)