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(deps): bump github.com/timshannon/bolthold to v0.0.0-20240314194003-30aac6950928 #2248

Merged
merged 8 commits into from
Jul 24, 2024

Conversation

harryzcy
Copy link
Contributor

Dependabot does not seem to bump this

@harryzcy harryzcy requested a review from a team as a code owner March 12, 2024 22:18
@harryzcy harryzcy changed the title Bump github.com/timshannon/bolthold to v0.0.0-20231129192944-dca5178aa629 build(deps): bump github.com/timshannon/bolthold to v0.0.0-20231129192944-dca5178aa629 Mar 12, 2024
Copy link
Contributor

mergify bot commented Apr 22, 2024

@harryzcy this pull request is now in conflict 😩

@mergify mergify bot added the conflict PR has conflicts label Apr 22, 2024
@harryzcy harryzcy changed the title build(deps): bump github.com/timshannon/bolthold to v0.0.0-20231129192944-dca5178aa629 build(deps): bump github.com/timshannon/bolthold to v0.0.0-20240314194003-30aac6950928 Apr 22, 2024
@mergify mergify bot removed the conflict PR has conflicts label Apr 22, 2024
wolfogre
wolfogre previously approved these changes Apr 22, 2024
Copy link
Contributor

mergify bot commented Apr 22, 2024

@harryzcy this pull request has failed checks 🛠

@mergify mergify bot added the needs-work Extra attention is needed label Apr 22, 2024
Copy link
Contributor

mergify bot commented May 6, 2024

@harryzcy this pull request is now in conflict 😩

@mergify mergify bot added the conflict PR has conflicts label May 6, 2024
@mergify mergify bot removed the conflict PR has conflicts label May 6, 2024
ChristopherHX
ChristopherHX previously approved these changes May 6, 2024
Copy link
Contributor

@ChristopherHX ChristopherHX left a comment

Choose a reason for hiding this comment

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

...actually this should be merged

Copy link

codecov bot commented May 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.44%. Comparing base (5a80a04) to head (aee4a35).
Report is 94 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #2248       +/-   ##
===========================================
+ Coverage   61.56%   76.44%   +14.88%     
===========================================
  Files          53       61        +8     
  Lines        9002     7919     -1083     
===========================================
+ Hits         5542     6054      +512     
+ Misses       3020     1308     -1712     
- Partials      440      557      +117     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mergify mergify bot removed the needs-work Extra attention is needed label May 7, 2024
Copy link
Contributor

mergify bot commented May 13, 2024

@harryzcy this pull request is now in conflict 😩

@mergify mergify bot added the conflict PR has conflicts label May 13, 2024
@harryzcy
Copy link
Contributor Author

merge conflicts fixed

@mergify mergify bot removed the conflict PR has conflicts label Jul 21, 2024
@mergify mergify bot merged commit 2feff3f into nektos:master Jul 24, 2024
11 checks passed
@harryzcy harryzcy deleted the bump-bolthold branch July 26, 2024 03:19
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.

3 participants