Skip to content

Comments

🌱 Bump go-github to v82#4923

Merged
spencerschrock merged 2 commits intoossf:mainfrom
Kielek:bump-go-github
Feb 2, 2026
Merged

🌱 Bump go-github to v82#4923
spencerschrock merged 2 commits intoossf:mainfrom
Kielek:bump-go-github

Conversation

@Kielek
Copy link
Contributor

@Kielek Kielek commented Jan 30, 2026

What kind of change does this PR introduce?

(Is it a bug fix, feature, docs update, something else?)

What is the current behavior?

Lack of support for immutable releases in go-github.

What is the new behavior (if this is a feature change)?**

  • Tests for the changes have been added (for bug fixes/features) Tests passes without significant changes

Which issue(s) this PR fixes

Towards #4823

Special notes for your reviewer

It updates go-github to the latest release. One of the releases between v53 a v82 brings support for immutable releases.
It should allow implement #4823.
Only one breaking change detected during compilation.

Does this PR introduce a user-facing change?

No

@Kielek Kielek marked this pull request as ready for review January 30, 2026 10:25
@Kielek Kielek requested a review from a team as a code owner January 30, 2026 10:25
@Kielek Kielek requested review from raghavkaul and spencerschrock and removed request for a team January 30, 2026 10:25
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jan 30, 2026
Signed-off-by: Piotr Kiełkowicz <pkielkow@cisco.com>
@codecov
Copy link

codecov bot commented Jan 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.69%. Comparing base (353ed60) to head (d9f319c).
⚠️ Report is 313 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4923      +/-   ##
==========================================
+ Coverage   66.80%   69.69%   +2.88%     
==========================================
  Files         230      251      +21     
  Lines       16602    15653     -949     
==========================================
- Hits        11091    10909     -182     
+ Misses       4808     3871     -937     
- Partials      703      873     +170     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@spencerschrock
Copy link
Member

Struggling with some unrelated gitlab e2e failures. I'll try merging this again in a few hours

@spencerschrock spencerschrock enabled auto-merge (squash) February 2, 2026 14:57
@spencerschrock spencerschrock merged commit ade5c77 into ossf:main Feb 2, 2026
37 checks passed
@Kielek Kielek deleted the bump-go-github branch February 16, 2026 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants