KEMBAR78
feat: dont lint backup files by mstruebing · Pull Request #367 · dotenv-linter/dotenv-linter · GitHub
Skip to content

Conversation

mstruebing
Copy link
Member

@mstruebing mstruebing commented Feb 3, 2021

also:
* add .bak extension to backup files

What are your thoughts on this solution @dotenv-linter/core - is this the right way to do it?

closes #359

✔ Checklist:

  • Commit messages have been written in Conventional Commits format;
  • This PR has been added to CHANGELOG.md (at the top of the list);
  • Tests for the changes have been added (for bug fixes / features);
  • Docs have been added / updated on the dotenv-linter.github.io (for bug fixes / features).

@mstruebing mstruebing force-pushed the feat/359/dont-lint-backup-files branch from dbf2536 to 8dac94e Compare February 3, 2021 14:51
Copy link
Member

@mgrachev mgrachev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mstruebing Thank you for your contribution! 👍

I have left a comment. Please take a look at it 👀

@mstruebing mstruebing force-pushed the feat/359/dont-lint-backup-files branch 2 times, most recently from 7814ee8 to 3dbf23c Compare February 4, 2021 11:33
also:
    * add `.bak` extension to backup files

closes #359
@mstruebing mstruebing force-pushed the feat/359/dont-lint-backup-files branch from 3dbf23c to abc1957 Compare February 8, 2021 14:17
@mgrachev mgrachev merged commit 3dadac6 into master Feb 8, 2021
@mgrachev mgrachev deleted the feat/359/dont-lint-backup-files branch February 8, 2021 15:09
@mgrachev
Copy link
Member

mgrachev commented Feb 8, 2021

@mstruebing Thank you! ✌️

@mgrachev mgrachev mentioned this pull request Mar 15, 2021
24 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

v3 recursively lints backup files by default

2 participants