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

docs: note about .yamllint file #682

Closed
wants to merge 1 commit into from

Conversation

erlangparasu
Copy link

No description provided.

@erlangparasu erlangparasu changed the title docs: note about .yamlline file docs: note about .yamllint file Aug 12, 2024
@coveralls
Copy link

Coverage Status

coverage: 99.825%. remained the same
when pulling 0af57c6 on erlangparasu:patch-1
into 99fb29a on adrienverge:master.

@adrienverge
Copy link
Owner

Hello, such new mention should come with details about other configuration locations. But this isn't really needed, because all this is already stated at the beginning of the documentation about configuration.

@erlangparasu
Copy link
Author

as a new user i am used to reading the readme at the beginning, and earlier i was confused about what the default file name for config was. so i intend to add this so that maybe there are new users so they are not confused at the beginning.

but if you think this is not necessary, just ignore it.

@erlangparasu erlangparasu deleted the patch-1 branch August 19, 2024 04:24
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 this pull request may close these issues.

3 participants