Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: Bump pre-commit lib versions #318

Merged
merged 1 commit into from Oct 27, 2021
Merged

Conversation

yangchoo
Copy link
Contributor

@yangchoo yangchoo commented Oct 19, 2021

Bumps all pre-commit libraries except pylint to their latest versions.
There are no breaking changes introduced here.

@yangchoo yangchoo force-pushed the yang-bump-pre-commit-versions branch from c808663 to 7c69ad7 Compare October 19, 2021 08:29
@yangchoo yangchoo requested review from joshhacksthings and EribertoLopez and removed request for joshhacksthings October 19, 2021 08:29
@codecov-commenter
Copy link

codecov-commenter commented Oct 19, 2021

Codecov Report

Merging #318 (5c956fd) into master (affb4be) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #318   +/-   ##
=======================================
  Coverage   82.30%   82.30%           
=======================================
  Files          20       20           
  Lines        3962     3962           
=======================================
  Hits         3261     3261           
  Misses        701      701           

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 affb4be...5c956fd. Read the comment docs.

@yangchoo yangchoo force-pushed the yang-bump-pre-commit-versions branch 2 times, most recently from 5c956fd to f8ec83e Compare October 20, 2021 14:23
EribertoLopez
EribertoLopez previously approved these changes Oct 20, 2021
Copy link
Contributor

@EribertoLopez EribertoLopez left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@yangchoo
Copy link
Contributor Author

@EribertoLopez would appreciate a re-review. Feel free to merge after accepting.

Had to repush due to changelog conflicts

Bumps all pre-commit libraries except pylint to their latest versions.
There are no breaking changes introduced here.
@yangchoo yangchoo force-pushed the yang-bump-pre-commit-versions branch from 04aa673 to 9e80032 Compare October 26, 2021 23:28
@yangchoo yangchoo merged commit 976604b into master Oct 27, 2021
@yangchoo yangchoo deleted the yang-bump-pre-commit-versions branch October 27, 2021 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants