-
-
Notifications
You must be signed in to change notification settings - Fork 241
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
secretlint documentation bug re: no .secretlintignore file #3328
Comments
Default .secreetlintignire that is shipped with MegaLinter image is not .gitignore but a default file containing https://github.com/oxsecurity/megalinter/blob/main/TEMPLATES/.secretlintignore |
@nvuillam As far as I got the description above and the code, the issue is that when I have no The default
which is likely part of To me, it could make sense
|
This issue has been automatically marked as stale because it has not had recent activity. If you think this issue should stay open, please remove the |
@pjungermann sorry for the delay (very long days at work ^^) I agree with your solution, would you like to make a PR ? :) |
FYI: relates to #1207 |
Removes the default `.secretlintignore` that was actually never applied. If `.secretlintignore` is absent, `.gitignore` is used instead. Removing this default config will adjust the misleading documentation as well. Fixes oxsecurity#3328
Removes the default `.secretlintignore` that was actually never applied. If `.secretlintignore` is absent, `.gitignore` is used instead. Removing this default config will adjust the misleading documentation as well. Fixes oxsecurity#3328
Removes the default `.secretlintignore` that was actually never applied. If `.secretlintignore` is absent, `.gitignore` is used instead. Removing this default config will adjust the misleading documentation as well. Fixes oxsecurity#3328
Describe the bug
I was wondering about the behavior of
secretlint
and looked at #2992. It led me to see that there is a difference between implementation and documentation.The documentation at https://megalinter.io/latest/descriptors/repository_secretlint/ says:
Expected behavior
To me, this implies that if I do not configure a
.secretlintignore
file, then a default one will be generated on the fly with the same contents as at that link, then the file will be discarded.Actual behavior
However, the actual code shows that if a custom
.secretlintignore
file is not found, then.gitignore
will be used, not a default.Desired outcome
Update the documentation to match the implementation.
The text was updated successfully, but these errors were encountered: