Skip to content
This repository was archived by the owner on Sep 11, 2025. It is now read-only.

Conversation

@kevinmingtarja
Copy link
Contributor

@kevinmingtarja kevinmingtarja commented Apr 10, 2025

Description

This PR replaces the previous approach of using the GitHub REST API (which was prone to rate-limiting), to using git ls-remote.

Checklist

All PRs should check the following boxes:

  • I have given this PR a title using the
    Conventional Commits syntax, leading with ,
    , , , etc.
    • The title should also be used for the commit message when the PR is squashed and merged.
  • I have formatted and linted my code with Trunk, per the instructions in
    the contributing guide.

If the PR includes a code change, then also check the following boxes. (If not, then delete the
next section.)

  • I have added an entry to the file.
    • Add to the UNRELEASED section at the top of the file, creating one if it doesn't yet exist.
    • Be sure to include the link to this PR, and please sort the section numerically by PR number.
  • I have manually tested the new or modified code, and it appears to behave correctly.
  • I have added or updated unit tests where appropriate, if applicable.

@kevinmingtarja kevinmingtarja requested review from a team and Copilot April 10, 2025 20:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

@kevinmingtarja kevinmingtarja enabled auto-merge (squash) April 10, 2025 20:51
@kevinmingtarja kevinmingtarja merged commit c2a1295 into main Apr 11, 2025
6 checks passed
@kevinmingtarja kevinmingtarja deleted the kevinm/release-info branch April 11, 2025 18:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants