Skip to content

fix!: Typo in field names in the PullStats struct - #3445

Merged
gmlewis merged 1 commit into
google:masterfrom
alexandear-org:fix/typo-pullstats-fields
Jan 20, 2025
Merged

fix!: Typo in field names in the PullStats struct#3445
gmlewis merged 1 commit into
google:masterfrom
alexandear-org:fix/typo-pullstats-fields

Conversation

@alexandear

Copy link
Copy Markdown
Contributor

BREAKING CHANGE: MergablePulls=>MergeablePulls, UnmergablePulls=>UnmergeablePulls

This PR corrects field typos in the github.PullStats struct:

  • MergablePulls -> MergeablePulls (because json:"mergeable_pulls")
  • UnmergablePulls -> UnmergeablePulls (because json:"unmergeable_pulls")

@gmlewisgmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @alexandear !
LGTM.

Awaiting second LGTM+Approval from any other contributor to this repo before merging.

@stevehipwell - do you have time for a code review?

@gmlewisgmlewis added NeedsReview PR is awaiting a review before merging. Breaking API Change PR will require a bump to the major version num in next release. Look here to see the change(s). labels Jan 20, 2025
@codecov

codecovBot commented Jan 20, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.26%. Comparing base (eb92cca) to head (e4c5f92).
Report is 2 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@## master #3445 +/- ##
=======================================
Coverage 92.26% 92.26% =======================================
Files 174 174 Lines 15023 15023 =======================================
Hits 13861 13861 Misses 1068 1068 Partials 94 94 

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

@stevehipwellstevehipwell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@gmlewis

Copy link
Copy Markdown
Collaborator

Thank you, @stevehipwell!
Merging.

@alexandearalexandear changed the title fix: Typo in field names in the PullStats structfix!: Typo in field names in the PullStats structJan 20, 2025
@gmlewis
gmlewis merged commit b8c2253 into google:masterJan 20, 2025
@gmlewisgmlewis removed the NeedsReview PR is awaiting a review before merging. label Jan 20, 2025
jlaportebot added a commit to jlaportebot/go-github that referenced this pull request Jun 28, 2026
BREAKING CHANGE: `MergablePulls`=>`MergeablePulls`, `UnmergablePulls`=>`UnmergeablePulls`
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Breaking API ChangePR will require a bump to the major version num in next release. Look here to see the change(s).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@alexandear@gmlewis@stevehipwell