Skip to content

Add CI to dry run repositories.yaml changes - #1097

Merged
badboy merged 18 commits into
mainfrom
benwu/dryrun-ci
Oct 27, 2026
Merged

badboy merged 18 commits into
mainfrom
benwu/dryrun-ci

Conversation

@BenWu

@BenWu BenWu commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Adds CI workflow to diff dry run outputs for main and the PR branch when repositories.yaml is changed, and posts a single comment that gets updated on each commit.

The dry run runs for forked PRs but the comment does not, so you would need to go to the workflow run page to see the diff. It's likely fine to run for forks but this is easier to reason about the security so I would rather leave that out for now.

@BenWu BenWu changed the title Benwu/dryrun ci Add CI to dry run repositories.yaml changes Oct 22, 2026
@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

probe-scraper output diff

Changes to the Glean metadata probe-scraper would publish with this PR, compared to the base branch. Metrics, pings and tags are not scraped, so the per-app metrics, pings and tags files are empty here. For example, a new metrics_files entry or ping moz_pipeline_metadata shows up in glean/repositories, but not its effect on those files.

No changes to probe-scraper output.


Generated for c768276 by this workflow run at 2026-10-26 19:59 UTC.

@BenWu
BenWu marked this pull request as ready for review October 26, 2026 20:05
@BenWu
BenWu requested a review from badboy October 26, 2026 20:06
@badboy
badboy merged commit a5171b4 into main Oct 27, 2026
11 checks passed
@badboy
badboy deleted the benwu/dryrun-ci branch October 27, 2026 09:21
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.

2 participants