Skip to content

Bot support for github revisions#3254

Merged
Archaeopteryx merged 16 commits into
mozilla:masterfrom
vrigal:github-bot-support
Apr 29, 2026
Merged

Bot support for github revisions#3254
Archaeopteryx merged 16 commits into
mozilla:masterfrom
vrigal:github-bot-support

Conversation

@vrigal
Copy link
Copy Markdown
Collaborator

@vrigal vrigal commented Mar 11, 2026

Depends #3252

Analysis of a github PR requires the --github-repository=<path> to be set, linking to a local clone set up at the infra level.
I tested with TRY_TASK_ID=c9G2pWZiRm2bJsQ8ZRJNow and TRY_TASK_GROUP_ID=Gf_KQB-EQQSGNwIY5iV-gg.

@vrigal vrigal force-pushed the github-bot-support branch 4 times, most recently from 718454e to fde2710 Compare March 11, 2026 14:52
This was referenced Mar 11, 2026
@marco-c
Copy link
Copy Markdown
Collaborator

marco-c commented Apr 15, 2026

@vrigal looks like tests are failing

@vrigal vrigal force-pushed the github-bot-support branch from 29b76c2 to 2a7eea0 Compare April 21, 2026 16:21
Comment thread backend/code_review_backend/issues/serializers.py
@La0 La0 requested a review from marco-c April 22, 2026 10:48
@La0 La0 assigned Archaeopteryx and vrigal and unassigned Archaeopteryx Apr 22, 2026
@La0 La0 requested a review from Archaeopteryx April 22, 2026 10:48
@La0
Copy link
Copy Markdown
Collaborator

La0 commented Apr 22, 2026

I pushed a few patches to use the same naming convention for changesets & repos as on Phabricator.

This is now capable of loading issues found on a github PR:

export TRY_RUN_ID=0
export TRY_TASK_ID="IsqhgJyuQq2g2M6hmVoTag"
export TRY_TASK_GROUP_ID="TbyUh4mPQb-vlvzXOR9Vrw"
code-review-bot --conf=./conf-dev.yml --mercurial-repository=./cache/

But this lacks the actual git clone to build hashes and thus populate the backend with issues (only diff + revision is created now)

@La0
Copy link
Copy Markdown
Collaborator

La0 commented Apr 22, 2026

Now the bot is able to clone base repo, set head repo as ref and checkout the precise revision.

Issues are finally published in the backend !

Comment thread bot/code_review_bot/__init__.py
@Archaeopteryx Archaeopteryx merged commit 84109a8 into mozilla:master Apr 29, 2026
11 checks passed
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.

4 participants