-
Notifications
You must be signed in to change notification settings - Fork 155
Description
Summary
The deployed website at https://git-proxy.finos.org does not reflect the current state of the main branch. Netlify deployments appear to run, but the site content is stale.
Root Cause
The netlify.toml file is missing critical build configuration. It currently only contains an ignore directive:
[build]
ignore = "git diff --quiet $CACHED_COMMIT_REF $COMMIT_REF . ../website/"This is missing:
base— Netlify doesn't know the website source is inwebsite/command— Netlify doesn't know to runnpm run build(Docusaurus)publish— Netlify doesn't know the built output is inwebsite/build/
Without these, Netlify either skips the build entirely (the ignore command succeeds because the paths resolve incorrectly without a base) or has no instructions to actually build and publish the site.
History
The configuration was originally correct (commit 0da90fb):
[build]
base = "./website"It was then replaced with the ignore-only approach (commit a1af8e7 and subsequent iterations), which removed base, command, and publish. The build/publish settings may have been configured directly in the Netlify dashboard at some point, but the ignore path ../website/ is relative to the repo root (since no base is set), which likely doesn't resolve correctly.
Proposed Fix
Update netlify.toml to include the full build configuration:
[build]
base = "website/"
command = "npm run build"
publish = "build/"
ignore = "git diff --quiet $CACHED_COMMIT_REF $COMMIT_REF . ../website/"Note: publish is relative to base, so build/ resolves to website/build/. The ignore command should also be validated — if base is set to website/, then ../website/ may need adjustment since ignore runs from the base directory.
Verification
After the fix:
- Push a change to
main(even a docs-only change) - Confirm Netlify triggers a build (not skipped)
- Confirm the build succeeds and the site updates
- Verify https://git-proxy.finos.org reflects the latest content