KEMBAR78
IncorrectDelimiter fixer (#246) by gillespiecd · Pull Request #258 · dotenv-linter/dotenv-linter · GitHub
Skip to content

Conversation

gillespiecd
Copy link
Contributor

@gillespiecd gillespiecd commented Aug 13, 2020

  1. Add IncorrectDelimiterFixer

Closes #246

I followed the same logic used in the checker, to keep it consistent.

✔ 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).

@gillespiecd gillespiecd force-pushed the delimiter-fixer branch 2 times, most recently from c82e24d to db901e8 Compare August 13, 2020 23:21
@codecov-commenter
Copy link

codecov-commenter commented Aug 13, 2020

Codecov Report

Merging #258 into master will decrease coverage by 1.41%.
The diff coverage is 98.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #258      +/-   ##
==========================================
- Coverage   97.17%   95.76%   -1.42%     
==========================================
  Files          26       27       +1     
  Lines        2055     2125      +70     
==========================================
+ Hits         1997     2035      +38     
- Misses         58       90      +32     
Impacted Files Coverage Δ
src/fixes/incorrect_delimiter.rs 98.55% <98.55%> (ø)
src/fixes.rs 94.91% <100.00%> (+0.04%) ⬆️
src/checks/duplicated_key.rs 80.50% <0.00%> (-19.50%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6005dbc...c33dabf. Read the comment docs.

@gillespiecd gillespiecd marked this pull request as ready for review August 13, 2020 23:27
DDtKey
DDtKey previously approved these changes Aug 14, 2020
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.

Thank you for your contribution! 🚀

@DDtKey DDtKey requested a review from a team August 14, 2020 08:44
@DDtKey DDtKey requested a review from a team August 14, 2020 10:10
@gillespiecd gillespiecd force-pushed the delimiter-fixer branch 4 times, most recently from 17775b3 to 500d693 Compare August 20, 2020 15:25
@gillespiecd gillespiecd force-pushed the delimiter-fixer branch 8 times, most recently from 789f720 to 4806ad2 Compare August 20, 2020 23:58
@mgrachev mgrachev requested a review from a team August 21, 2020 13:33
@mgrachev mgrachev modified the milestone: v2.2.0 Aug 21, 2020
@gillespiecd gillespiecd force-pushed the delimiter-fixer branch 2 times, most recently from 98061e6 to 86bf6f8 Compare August 21, 2020 16:14
@mgrachev mgrachev requested review from a team and DDtKey August 22, 2020 10:55
@gillespiecd gillespiecd force-pushed the delimiter-fixer branch 2 times, most recently from e04edbf to ad32989 Compare August 22, 2020 15:40
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.

LGTM 👍

@mgrachev mgrachev merged commit 967fa7d into dotenv-linter:master Aug 23, 2020
@mgrachev
Copy link
Member

@gillespiecd Great job! Thank you! ❤️

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.

Add fixer: IncorrectDelimiterFixer

5 participants