Skip to content

Index definitions for type-search on all branches#86

Merged
ChrisPenner merged 26 commits into
mainfrom
cp/branch-definition-search
Jul 3, 2025
Merged

Index definitions for type-search on all branches#86
ChrisPenner merged 26 commits into
mainfrom
cp/branch-definition-search

Conversation

@ChrisPenner
Copy link
Copy Markdown
Member

@ChrisPenner ChrisPenner commented May 29, 2025

Overview

  • TODO fix codebase owner calculation for contributor branches in the DB trigger

Adds the mechanism to index arbitrary branches for definition search, not just releases.

Implementation notes

  • Adds a new definition search indexing queue that operates based on the root branch hash rather than release
  • Add triggers to project branches and releases to add to this queue
  • Always index branch and release pushes, but also copy releases into the global search queue when relevant.
  • TODO Add new endpoints for searching within a project branch.

Test coverage

  • Existing transcripts
  • New transcripts for project branch searching

@ChrisPenner ChrisPenner force-pushed the cp/branch-definition-search branch 2 times, most recently from 88b8286 to 8bbd821 Compare June 3, 2025 17:56
@ChrisPenner ChrisPenner force-pushed the cp/branch-definition-search branch from 8bbd821 to eb216f0 Compare July 1, 2025 21:36
@ChrisPenner ChrisPenner force-pushed the cp/branch-definition-search branch from a71d54a to 55c0be8 Compare July 2, 2025 17:31
@ChrisPenner ChrisPenner marked this pull request as ready for review July 2, 2025 17:42
@ChrisPenner ChrisPenner force-pushed the cp/branch-definition-search branch from 78c7e5f to dff7d51 Compare July 2, 2025 20:33
@ChrisPenner ChrisPenner force-pushed the cp/branch-definition-search branch from 4fb8cf4 to 917ce0a Compare July 3, 2025 00:04
@ChrisPenner ChrisPenner merged commit 917ce0a into main Jul 3, 2025
8 checks passed
@ChrisPenner ChrisPenner deleted the cp/branch-definition-search branch July 3, 2025 16:46
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.

1 participant