KEMBAR78
refactor(cli): get rid of `list` command by mgrachev · Pull Request #619 · dotenv-linter/dotenv-linter · GitHub
Skip to content

Conversation

mgrachev
Copy link
Member

@mgrachev mgrachev commented Jan 2, 2023

It was moved to help:

Lightning-fast linter for .env files

Usage: dotenv-linter [OPTIONS] [input]... [COMMAND]

Commands:
  compare  Compares if files have the same keys [aliases: c]
  fix      Automatically fixes warnings [aliases: f]

Arguments:
  [input]...  files or paths

Options:
  -e, --exclude [<FILE_NAME>...]  Excludes files from check
  -s, --skip [<CHECK_NAME>...]    Skips checks [possible values: DuplicatedKey, EndingBlankLine, ExtraBlankLine, IncorrectDelimiter, KeyWithoutValue, LeadingCharacter, LowercaseKey, QuoteCharacter, SpaceCharacter, SubstitutionKey, TrailingWhitespace, UnorderedKey, ValueWithoutQuotes]
  -r, --recursive                 Recursively searches and checks .env files
      --no-color                  Turns off the colored output
  -q, --quiet                     Doesn't display additional information
      --not-check-updates         Doesn't check for updates
  -h, --help                      Print help information
  -V, --version                   Print version information

✔ Checklist:

  • Commit messages have been written in Conventional Commits format;
  • 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).

@codecov-commenter
Copy link

codecov-commenter commented Jan 2, 2023

Codecov Report

Merging #619 (fb6047e) into master (46f5ad2) will increase coverage by 0.11%.
The diff coverage is 98.90%.

@@            Coverage Diff             @@
##           master     #619      +/-   ##
==========================================
+ Coverage   98.99%   99.11%   +0.11%     
==========================================
  Files          43       44       +1     
  Lines        2792     2814      +22     
==========================================
+ Hits         2764     2789      +25     
+ Misses         28       25       -3     
Impacted Files Coverage Δ
src/checks.rs 100.00% <ø> (ø)
src/cli/options.rs 96.55% <96.55%> (ø)
src/cli/mod.rs 97.72% <100.00%> (ø)
src/common/lint_kind.rs 100.00% <100.00%> (ø)
src/lib.rs 99.47% <100.00%> (+0.45%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@mgrachev mgrachev force-pushed the remove-list-command branch 2 times, most recently from 5d7aa4f to e7cd70c Compare January 2, 2023 13:55
Signed-off-by: Mikhail Grachev <work@mgrachev.com>
@mgrachev mgrachev force-pushed the remove-list-command branch from e7cd70c to fb6047e Compare January 2, 2023 14:03
@mgrachev mgrachev merged commit c00515e into master Jan 2, 2023
@mgrachev mgrachev deleted the remove-list-command branch January 2, 2023 14:29
@DDtKey
Copy link
Member

DDtKey commented Jan 2, 2023

@mgrachev
Good one, but we probably need to make next release as major one due to removing the CLI command.

@mgrachev
Copy link
Member Author

mgrachev commented Jan 2, 2023

@mgrachev Good one, but we probably need to make next release as major one due to removing the CLI command.

Yes, agree 👍

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