KEMBAR78
Move show-checks flag to main.rs by mgrachev · Pull Request #227 · dotenv-linter/dotenv-linter · GitHub
Skip to content

Conversation

@mgrachev
Copy link
Member

What have been done:

  • Sorting is removed from the available_check_names method, because all checks are already in alphabetical order;
  • The show checks flag is moved to main.rs file, because it allowed to exit from the program.

✔ Checklist:

  • This PR has been added to CHANGELOG.md (at the top of the list).

@mgrachev mgrachev requested a review from a team June 29, 2020 14:53
@codecov-commenter
Copy link

Codecov Report

Merging #227 into master will decrease coverage by 0.07%.
The diff coverage is 73.91%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #227      +/-   ##
==========================================
- Coverage   96.66%   96.58%   -0.08%     
==========================================
  Files          16       16              
  Lines        1499     1495       -4     
==========================================
- Hits         1449     1444       -5     
- Misses         50       51       +1     
Impacted Files Coverage Δ
src/lib.rs 85.71% <25.00%> (-1.06%) ⬇️
src/main.rs 80.00% <83.33%> (ø)
src/checks.rs 100.00% <100.00%> (ø)

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 fa83975...f211097. Read the comment docs.

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.

I apologize, mistakenly accepted the suggestions and had to roll back.

@DDtKey DDtKey force-pushed the main-refactoring branch from f211097 to 3abb4ce Compare June 29, 2020 18:02
@DDtKey
Copy link
Member

DDtKey commented Jun 29, 2020

It’s a good idea to leave terminal operations in main.rs! 🚀
Sorry for some mess with suggestions for improvement.

Signed-off-by: Mikhail Grachev <work@mgrachev.com>
Copy link
Contributor

@wesleimp wesleimp left a comment

Choose a reason for hiding this comment

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

LGTM 👌

Copy link
Member

@mstruebing mstruebing left a comment

Choose a reason for hiding this comment

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

I like it :)

@mgrachev mgrachev merged commit 6f81f1b into master Jun 30, 2020
@mgrachev mgrachev deleted the main-refactoring branch June 30, 2020 09:12
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.

5 participants