From 5de4a7e75bbaf9c441757499efc075bcc93d9ee1 Mon Sep 17 00:00:00 2001 From: Gerhard Lazu Date: Thu, 10 Feb 2022 11:00:38 +0000 Subject: [PATCH] Always build the docs website If we trigger a deploy, then we want the deploy to actually run. I am assuming that this was in place when there was a single GitHub Actions workflow that would trigger this all the time, even when nothing changed in `docs/**`. Since https://github.com/dagger/dagger/pull/1498, we no longer have this problem. After merging https://github.com/dagger/dagger/pull/1591, I was caught by this behaviour which to me seemed surprising. This change should make docs deploy via Netlify behave in a more predictable way. Signed-off-by: Gerhard Lazu --- website/netlify.toml | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/website/netlify.toml b/website/netlify.toml index 86fa5dee..ba17fe5c 100644 --- a/website/netlify.toml +++ b/website/netlify.toml @@ -4,18 +4,8 @@ environment = { NODE_VERSION = "16.13.1" } [context.production] - # Do not build the site if there's no site-related changes since the last - # commit. - # This assumes PRs to `main` only contain merge commits, so it's safe to - # compare the current commit to the previous. - # We can't use `$CACHED_COMMIT_REF` because that points to the PR preview - # that was built before merging. - ignore = "git diff --quiet HEAD^ HEAD -- . ../docs/" [context.deploy-preview] - # Do not build the site if there's no site-related changes since the last - # deploy. - ignore = "git diff --quiet $COMMIT_REF $CACHED_COMMIT_REF -- ./ ../docs/" command = "yarn build:withoutAuth" [[redirects]]