Skip to content

馃悰 Fix pagination issue in getLatestRelease (#651) - #652

Merged
WtfJoke merged 1 commit into
mainfrom
fixLatestReleaseNotWorking
Apr 13, 2026
Merged

WtfJoke merged 1 commit into
mainfrom
fixLatestReleaseNotWorking

Conversation

@WtfJoke

@WtfJoke WtfJoke commented Apr 13, 2026

Copy link
Copy Markdown
Owner

When there are too many tectonic GitHub releases (for example a lot of component package (e.g., tectonic_xetex_layout, tectonic_bundles, etc.. releases), the actual tectonic tectonic@ release gets pushed past the default 30 per_page limit of the listReleases API.

Use a hybrid approach:

@WtfJoke
WtfJoke force-pushed the fixLatestReleaseNotWorking branch from 5ba483c to 12643a4 Compare April 13, 2026 21:22
@WtfJoke
WtfJoke enabled auto-merge April 13, 2026 21:22
When there are too many tectonic GitHub releases (for example a lot of component package (e.g., tectonic_xetex_layout, tectonic_bundles, etc.. releases), the actual tectonic `tectonic@` release gets pushed past the default 30 `per_page` limit of the `listReleases` [API](https://docs.github.com/en/rest/releases/releases?apiVersion=2026-03-10#list-releases).

Use a hybrid approach:
 - Try [getLatestRelease API](https://docs.github.com/en/rest/releases/releases?apiVersion=2026-03-10#get-the-latest-release) first
 - Fallback to a paginated search with [listReleases API](https://docs.github.com/en/rest/releases/releases?apiVersion=2026-03-10#list-releases) with early termination and a `per_page size` of 100 (max)
@WtfJoke
WtfJoke force-pushed the fixLatestReleaseNotWorking branch from 12643a4 to 75fccc5 Compare April 13, 2026 21:28
@WtfJoke WtfJoke linked an issue Apr 13, 2026 that may be closed by this pull request
@sonarqubecloud

Copy link
Copy Markdown

@WtfJoke
WtfJoke disabled auto-merge April 13, 2026 21:31
@WtfJoke
WtfJoke merged commit c01e257 into main Apr 13, 2026
11 checks passed
@WtfJoke
WtfJoke deleted the fixLatestReleaseNotWorking branch April 13, 2026 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Couldn't get the latest Tectonic release since recent update

1 participant