-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: prevent unnecessary netlify builds #8755
Conversation
✅ Deploy Preview for vite-docs-main ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@bluwy for some reason we are getting every build canceled in netlify 🤔
|
@patak-dev is there a way to see the full deploy logs (and history)? Looks like main.vitejs.dev is deploying fine (its deploy) |
That seems to be the only logs. PR: #9390
It is the same config from the repo POV (https://github.com/vitejs/vite/blob/main/netlify.toml), maybe |
Yeah I guess |
Or maybe we can retry again when we merge #9379. Perhaps it's a fluke after some Netlify configuration. |
I also noticed the changes for #9390 haven't reflected in docs, yet. Is this the reason why? @patak-dev |
Just checked with #9454, and the deploy was still canceled with:
I'm out of ideas. Maybe we could ask Evan to recreate the site in netlify. Blu you can ping him or I'll do so when I'm back if this hasnt been resolved yet. |
Description
Only rebuild on netlify when
docs/**/*
,package.json
,pnpm-lock.yaml
, ornetlify.toml
changed.Additional context
A bit annoyed of the netlify build message in each PR 😅
What is the purpose of this pull request?
Before submitting the PR, please make sure you do the following
fixes #123
).