KEMBAR78
Fix a bug with UnorderedKeyChecker and control comments by mgrachev · Pull Request #283 · dotenv-linter/dotenv-linter · GitHub
Skip to content

Conversation

mgrachev
Copy link
Member

@mgrachev mgrachev commented Sep 19, 2020

I have found a bug in this PR #281.

If we run dotenv-linter against this file:

# .env
BB=1
# dotenv-linter:off LowercaseKey
Aa=B
X=X

We will get the warning:

$ dotenv-linter .env
.env:3 UnorderedKey: The Aa key should go before the BB key

@evgeniy-r Please take a look at it 👀

✔ Checklist:

  • 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 (for bug fixes / features).

Signed-off-by: Mikhail Grachev <work@mgrachev.com>
@mgrachev mgrachev force-pushed the fix-bug-with-unordered-key branch from 1d59ee8 to 2829eb5 Compare September 19, 2020 11:22
@mgrachev mgrachev added this to the v2.2.0 milestone Sep 19, 2020
@mgrachev mgrachev requested review from a team and DDtKey September 19, 2020 11:27
@mgrachev mgrachev marked this pull request as ready for review September 19, 2020 11:28
Copy link
Member

@DDtKey DDtKey left a comment

Choose a reason for hiding this comment

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

Really good catch 👍
We shouldn't skip comments for UnorderedKeyChecker now.
Thank you!

@evgeniy-r
Copy link
Contributor

You are absolutely right 👍

@evgeniy-r
Copy link
Contributor

I will rebase my PR about the UnorderedKey fixer, when this be merged.

@mgrachev mgrachev merged commit 5974a30 into master Sep 19, 2020
@mgrachev mgrachev deleted the fix-bug-with-unordered-key branch September 19, 2020 12:02
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.

3 participants