William Chargin 383f8d406e

deploy.sh: depend on build_static_site.sh (#611) ...

Summary:
This is a minimal patch to make the deploy script depend on the
separate, well-tested build script introduced in #592. The patch
suggests a number of improvements that can be made to the deploy script
itself, but I’m deferring those as they’re not on the critical path.
These are tracked at #610.

Test Plan:
Apply the below patch. Then run `./scripts/deploy.sh -n`, and verify
that the served output is correct, including the CNAME file. Verify that
when the `DEPLOY_CNAME_URL` default value is removed, the resulting
output does _not_ have a CNAME file.

Patch, for ease of building:

```diff
diff --git a/scripts/deploy.sh b/scripts/deploy.sh
index cddcece..8925e5b 100755
--- a/scripts/deploy.sh
+++ b/scripts/deploy.sh
@@ -23,7 +23,7 @@ main() {
     preview_dir=
     trap cleanup EXIT

-    ensure_clean_working_tree
+    #ensure_clean_working_tree
     build_and_deploy
 }

@@ -76,8 +76,7 @@ build_and_deploy() {
     "${sourcecred_repo}/scripts/build_static_site.sh" \
         --target "${static_site}" \
         ${DEPLOY_CNAME_URL:+--cname "${DEPLOY_CNAME_URL}"} \
-        --repo ipfs/js-ipfs \
-        --repo sourcecred/sourcecred \
+        --repo sourcecred/example-github \
         ;

     sourcecred_site="$(mktemp -d --suffix ".sourcecred-site")"
```

wchargin-branch: deploy-via-build

2018-08-06 19:00:35 -07:00

..

backend.js

Unify a command-line entry point module (#344)

2018-06-05 11:11:48 -07:00

build_static_site.sh

Create script to build static site (#592)

2018-08-06 13:05:40 -07:00

check-stopships.sh

Permit STOPSHIPs in Markdown files (#573)

2018-07-31 10:58:29 -07:00

deploy.sh

deploy.sh: depend on build_static_site.sh (#611)

2018-08-06 19:00:35 -07:00

ensure-flow.sh

Fix ensure-flow.sh running under Node (#314)

2018-05-29 12:20:53 -07:00