Skip to content

Fix the demo deploy: drop the configure-pages step - #5

Merged
YauhenBichel merged 1 commit into
mainfrom
fix/pages-deploy
Sep 26, 2026
Merged

YauhenBichel merged 1 commit into
mainfrom
fix/pages-deploy

Conversation

@YauhenBichel

Copy link
Copy Markdown
Member

The first two demo deploys failed at the configure-pages step with "Resource not accessible by integration".

Why: the build job only has contents: read, and configure-pages also needs pages: read.

Fix: remove the step. The demo already uses relative paths (Vite base: './'), so nothing needed its output. This keeps the build job's permissions as small as possible.

Checked

  • A manual run on this branch: build the demo now succeeds.
  • publish was refused on the branch. That's expected, because the github-pages environment only deploys from main.
  • After merging, the push to main deploys to https://molecare.github.io/react-photo-compare/.

🤖 Generated with Claude Code

It needs pages: read, which the build job does not have, so the first
deploy failed with 'Resource not accessible by integration'. The demo
uses relative paths, so the step was not needed at all.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@YauhenBichel
YauhenBichel merged commit 6890330 into main Sep 26, 2026
10 of 11 checks passed
@YauhenBichel
YauhenBichel deleted the fix/pages-deploy branch September 26, 2026 17:16

This branch had an error being deployed

1 failed deployment
github-pages — 57a4dfac Deployed Sep 26, 2026 by YauhenBichel via publish to GitHub Pages #3
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