KEMBAR78
test: add tests to verify cli application by mgrachev · Pull Request #490 · dotenv-linter/dotenv-linter · GitHub
Skip to content

Conversation

mgrachev
Copy link
Member

@mgrachev mgrachev commented Mar 17, 2022

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

@mgrachev mgrachev force-pushed the cli-tests branch 7 times, most recently from c0be6a2 to ba83985 Compare March 23, 2022 11:59
@codecov-commenter
Copy link

Codecov Report

Merging #490 (ba83985) into master (61a1928) will increase coverage by 0.00%.
The diff coverage is 83.33%.

@@           Coverage Diff           @@
##           master     #490   +/-   ##
=======================================
  Coverage   98.92%   98.92%           
=======================================
  Files          41       41           
  Lines        2698     2702    +4     
=======================================
+ Hits         2669     2673    +4     
  Misses         29       29           
Impacted Files Coverage Δ
src/checks/duplicated_key.rs 100.00% <ø> (ø)
src/common/comment.rs 100.00% <ø> (ø)
src/common/file_entry.rs 100.00% <ø> (ø)
src/common/output/check.rs 100.00% <ø> (ø)
src/common/output/fix.rs 87.87% <ø> (ø)
src/common/warning.rs 100.00% <ø> (ø)
src/fixes/duplicated_key.rs 100.00% <ø> (ø)
src/fs_utils.rs 95.40% <0.00%> (ø)
src/cli.rs 100.00% <100.00%> (ø)
src/lib.rs 98.99% <100.00%> (ø)
... and 3 more

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 61a1928...ba83985. Read the comment docs.

Signed-off-by: Mikhail Grachev <work@mgrachev.com>
@mgrachev mgrachev merged commit 7e54d56 into master Mar 23, 2022
@mgrachev mgrachev deleted the cli-tests branch March 23, 2022 13:41
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.

2 participants