Skip to content

docs: Add apiVersion to GitHub API link#4147

Open
alexandear wants to merge 1 commit intogoogle:masterfrom
alexandear-org:docs/include-api-version
Open

docs: Add apiVersion to GitHub API link#4147
alexandear wants to merge 1 commit intogoogle:masterfrom
alexandear-org:docs/include-api-version

Conversation

@alexandear
Copy link
Copy Markdown
Contributor

Since there are two API versions (see #4077), GitHub API links redirect to the latest version by default. However, this library currently supports version 2022-11-28, so we need to add ?apiVersion=2022-11-28 to every link.

@alexandear alexandear force-pushed the docs/include-api-version branch from 40764da to 16a1a93 Compare April 10, 2026 13:54
@alexandear alexandear force-pushed the docs/include-api-version branch from 16a1a93 to 27fb499 Compare April 10, 2026 14:00
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.75%. Comparing base (9ad687d) to head (27fb499).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4147   +/-   ##
=======================================
  Coverage   93.75%   93.75%           
=======================================
  Files         211      211           
  Lines       19687    19687           
=======================================
  Hits        18458    18458           
  Misses       1031     1031           
  Partials      198      198           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gmlewis gmlewis added the NeedsReview PR is awaiting a review before merging. label Apr 10, 2026
Copy link
Copy Markdown
Collaborator

@gmlewis gmlewis left a comment

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.

cc: @stevehipwell - @zyfy29 - @Not-Dhananjay-Mishra - @munlicode

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NeedsReview PR is awaiting a review before merging.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants