KEMBAR78
Integration test for fixers by evgeniy-r · Pull Request #309 · dotenv-linter/dotenv-linter · GitHub
Skip to content

Conversation

@evgeniy-r
Copy link
Contributor

@evgeniy-r evgeniy-r commented Oct 5, 2020

I added some different test examples.
It will be easy to add more.

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

@mgrachev mgrachev linked an issue Oct 6, 2020 that may be closed by this pull request
@mgrachev mgrachev added this to the v2.2.0 milestone Oct 6, 2020
@codecov-io
Copy link

codecov-io commented Oct 6, 2020

Codecov Report

Merging #309 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #309   +/-   ##
=======================================
  Coverage   96.98%   96.98%           
=======================================
  Files          32       32           
  Lines        2292     2292           
=======================================
  Hits         2223     2223           
  Misses         69       69           

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 e7dd376...77d9d9e. Read the comment docs.

@evgeniy-r evgeniy-r force-pushed the feat/294/test branch 2 times, most recently from 4bf2fb8 to 1808455 Compare October 8, 2020 20:31
@evgeniy-r evgeniy-r marked this pull request as ready for review October 8, 2020 20:40
@mgrachev mgrachev requested a review from a team October 9, 2020 07:53
@mgrachev
Copy link
Member

mgrachev commented Oct 9, 2020

@evgeniy-r Thank you for your help! ❤️

@mgrachev mgrachev merged commit feaee82 into dotenv-linter:master Oct 9, 2020
Anthuang pushed a commit to Anthuang/dotenv-linter that referenced this pull request Oct 12, 2020
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.

Write an integration test for all fixers

3 participants