KEMBAR78
refactor(cli): remove `file_entry.rs` as not inluded in file tree by brahms116 · Pull Request #657 · dotenv-linter/dotenv-linter · GitHub
Skip to content

Conversation

brahms116
Copy link
Contributor

Remove file_entry.rs which is no longer included in the file tree. Content has been moved inside the dotenv-lookup crate inside src/file.rs.

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

Remove file_entry.rs which is no longer included in the file tree.
Content has been moved inside the dotenv-lookup crate inside `src/file.rs`
@mgrachev mgrachev changed the title chore(cli): Remove file_entry.rs as not inluded in file tree refactor(cli): remove file_entry.rs as not inluded in file tree Apr 3, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #657 (62cc04f) into master (2f47ed1) will decrease coverage by 0.04%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##           master     #657      +/-   ##
==========================================
- Coverage   99.16%   99.13%   -0.04%     
==========================================
  Files          46       46              
  Lines        2765     2766       +1     
==========================================
  Hits         2742     2742              
- Misses         23       24       +1     

see 1 file with indirect coverage changes

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

@mgrachev mgrachev merged commit ca4e3fe into dotenv-linter:master Apr 4, 2023
@mgrachev
Copy link
Member

mgrachev commented Apr 4, 2023

@brahms116 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.

3 participants