KEMBAR78
Updated lifetimes in EndingBlankLineChecker by rossjones · Pull Request #197 · dotenv-linter/dotenv-linter · GitHub
Skip to content

Conversation

rossjones
Copy link
Contributor

@rossjones rossjones commented May 23, 2020

Changes the lifetimes from 'static to 'a in EndingBlankLineChecker
Fixes #185

✔ Checklist:

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

@codecov-commenter
Copy link

codecov-commenter commented May 23, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #197   +/-   ##
=======================================
  Coverage   96.71%   96.71%           
=======================================
  Files          14       14           
  Lines        1096     1096           
=======================================
  Hits         1060     1060           
  Misses         36       36           
Impacted Files Coverage Δ
src/checks/ending_blank_line.rs 91.42% <ø> (ø)

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 106ef61...f60fa8d. Read the comment docs.

mstruebing
mstruebing previously approved these changes May 23, 2020
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.

Nice work, thank you 🚀

Copy link
Member

@mgrachev mgrachev left a comment

Choose a reason for hiding this comment

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

@rossjones 👋Thank you for your contribution! 👍

I have left a comment. Please, fix it.

@rossjones rossjones force-pushed the 185-ending-blank-line-replace-static branch from e1c9e07 to 951edd1 Compare May 23, 2020 13:52
Fix the text of the pull request link in changelog.

Co-authored-by: Grachev Mikhail <work@mgrachev.com>
@mgrachev mgrachev merged commit a7cfb5f into dotenv-linter:master May 23, 2020
@mgrachev
Copy link
Member

@rossjones Great work! Thank you! ❤️

🙏If it’s not difficult for you, please support the project - click on the star on GitHub ⭐️

@rossjones rossjones deleted the 185-ending-blank-line-replace-static branch May 23, 2020 14:16
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.

EndingBlankLineChecker: Replace &'static str with &'a str

4 participants