Skip to content

Add skipPatterns to default no-more-404 block for production builds#4985

Closed
shannonbradshaw wants to merge 1 commit intoviamrobotics:mainfrom
shannonbradshaw:fix-prod-skip-patterns
Closed

Add skipPatterns to default no-more-404 block for production builds#4985
shannonbradshaw wants to merge 1 commit intoviamrobotics:mainfrom
shannonbradshaw:fix-prod-skip-patterns

Conversation

@shannonbradshaw
Copy link
Copy Markdown
Collaborator

Summary

Production deploy is failing with `Plugin "@eggnstone/netlify-plugin-no-more-404" failed: 80 paths missing`.

The operate-removal PR (#4983) added `skipPatterns` under `[context.deploy-preview.plugins.inputs]` and `[context.branch-deploy.plugins.inputs]` but missed the default `[plugins.inputs]` block used for production builds. The production deploy log shows `skipPatterns:` empty.

Test plan

  • Production deploy on merge succeeds (no "paths missing" error)

🤖 Generated with Claude Code

The operate-removal PR added skipPatterns to deploy-preview and
branch-deploy contexts but missed the default [plugins.inputs] block
used for production builds. Production deploy of the merged operate-
removal PR is failing with 80 missing paths.

Adding the same skipPatterns list to the default block fixes the
production build.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 20, 2026

Deploy Preview for viam-docs ready!

Name Link
🔨 Latest commit a757283
🔍 Latest deploy log https://app.netlify.com/projects/viam-docs/deploys/69e69016cbd4730008ef1238
😎 Deploy Preview https://deploy-preview-4985--viam-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 43 (🟢 up 4 from production)
Accessibility: 99 (no change from production)
Best Practices: 92 (no change from production)
SEO: 89 (no change from production)
PWA: 60 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@viambot viambot added the safe to build This pull request is marked safe to build from a trusted zone label Apr 20, 2026
@shannonbradshaw
Copy link
Copy Markdown
Collaborator Author

Replacing with proper redirects per feedback — skipPatterns silences the check but doesn't fix the 404 for users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to build This pull request is marked safe to build from a trusted zone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants