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

chore: update pre-commit config #3365

Merged
merged 1 commit into from
Nov 2, 2023
Merged

chore: update pre-commit config #3365

merged 1 commit into from
Nov 2, 2023

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 1, 2023

Automated changes by create-pull-request GitHub action

@terriko
Copy link
Contributor

terriko commented Oct 16, 2023

Not sure what was up with this one and the test fail. I'm updating the branch to let the tests re-run but it's likely that I'll leave this un-merged until after hacktoberfest completes just so people don't have to update their pre-commit environment partway through a PR.

@codecov-commenter
Copy link

codecov-commenter commented Oct 16, 2023

Codecov Report

Merging #3365 (eeccca7) into main (f1525ce) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3365      +/-   ##
==========================================
- Coverage   80.20%   80.18%   -0.03%     
==========================================
  Files         758      758              
  Lines       11596    11596              
  Branches     1568     1568              
==========================================
- Hits         9301     9298       -3     
- Misses       1857     1859       +2     
- Partials      438      439       +1     
Flag Coverage Δ
longtests 74.82% <ø> (ø)
win-longtests 78.17% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@terriko
Copy link
Contributor

terriko commented Nov 1, 2023

closing and re-opening for tests

@terriko terriko closed this Nov 1, 2023
@terriko terriko reopened this Nov 1, 2023
Copy link
Contributor

@terriko terriko left a comment

Choose a reason for hiding this comment

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

Looks like this can be merged now.

@terriko terriko merged commit db96d6d into main Nov 2, 2023
21 checks passed
terriko pushed a commit to terriko/cve-bin-tool that referenced this pull request Nov 16, 2023
Co-authored-by: GitHub <noreply@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants