Skip to content

feat(gitea/forgejo): set poster server filter for pr cache#34374

Merged
viceice merged 2 commits intomainfrom
feat/gitea-pr-poster
Feb 21, 2025
Merged

feat(gitea/forgejo): set poster server filter for pr cache#34374
viceice merged 2 commits intomainfrom
feat/gitea-pr-poster

Conversation

@viceice
Copy link
Copy Markdown
Member

@viceice viceice commented Feb 21, 2025

Changes

  • set poster server filter for pr cache
  • optimize pr paging
  • fix pr api path

Context

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

- optimize pr paging
- fix pr api path
@viceice viceice requested review from rarkins and secustor February 21, 2025 12:36
rarkins
rarkins previously approved these changes Feb 21, 2025
Copy link
Copy Markdown
Member

@secustor secustor left a comment

Choose a reason for hiding this comment

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

else LGTM

Comment thread lib/modules/platform/gitea/pr-cache.ts Outdated
Comment thread lib/modules/platform/gitea/pr-cache.ts Outdated
Co-authored-by: Sebastian Poxhofer <secustor@users.noreply.github.com>
@viceice viceice added this pull request to the merge queue Feb 21, 2025
Merged via the queue into main with commit c7737be Feb 21, 2025
@viceice viceice deleted the feat/gitea-pr-poster branch February 21, 2025 13:56
@renovate-release
Copy link
Copy Markdown

🎉 This PR is included in version 39.177.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants