Skip to content

[test PR]#1003

Open
marcustyphoon wants to merge 2 commits into
mainfrom
ksjfsdjnfs
Open

[test PR]#1003
marcustyphoon wants to merge 2 commits into
mainfrom
ksjfsdjnfs

Conversation

@marcustyphoon

@marcustyphoon marcustyphoon commented Jan 7, 2026

Copy link
Copy Markdown
Member

Empty commit to test the deploy workflow. Do we get a cloudflare bot comment like #999 did? Did this suddenly change?

@github-actions

github-actions Bot commented Jan 7, 2026

Copy link
Copy Markdown
Contributor

Deployed build to Cloudflare!

Last commit: 9271e8f
Preview URL: https://58a68d2a.discretize-gear-optimizer.pages.dev
Branch Preview URL: https://ksjfsdjnfs.discretize-gear-optimizer.pages.dev

@marcustyphoon

marcustyphoon commented Jan 7, 2026

Copy link
Copy Markdown
Member Author

We do not. hm. may be a difference in default github_token permissions, may be something else.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying optimizer with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9271e8f
Status: ✅  Deploy successful!
Preview URL: https://58a68d2a.discretize-gear-optimizer.pages.dev
Branch Preview URL: https://ksjfsdjnfs.discretize-gear-optimizer.pages.dev

View logs

@marcustyphoon

Copy link
Copy Markdown
Member Author

Aha! Cloudflare automatically comments (presumably because the cloudflare github app is installed in this organization?) only if you deploy the PR head commit, not the PR merge commit. I assume they're firing code on any deployment that checks if the deployed commit is a PR head and comments on that PR if so.

Generally speaking, it makes sense to run CI on (test and deploy) the PR merge commit, so I think this is kind of a bug. Maybe I'll try to find where to report this.

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