Skip to content
This repository was archived by the owner on Jun 14, 2024. It is now read-only.

Conversation

chalin
Copy link
Collaborator

@chalin chalin commented Jul 29, 2021

Add a default value for DEPLOY_PRIME_URL (which isn't set when we run locally) -- otherwise the script just fails.

nate-double-u
nate-double-u previously approved these changes Jul 29, 2021
Copy link
Member

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated build:preview builds (but doesn't serve) locally,
Deploy preview looks good (https://deploy-preview-105--cncf-hugo-starter.netlify.app/),
Build logs look good (https://app.netlify.com/sites/cncf-hugo-starter/deploys/6102bd3b54759e0007abda1f)

/lgtm

@nate-double-u
Copy link
Member

Needs DCO before we can merge.

@chalin chalin force-pushed the chalin-local-build-preview-2021-07-29 branch from a9364a6 to 3924c2e Compare July 29, 2021 17:24
@chalin
Copy link
Collaborator Author

chalin commented Jul 29, 2021

Thanks for all the checks!

@chalin
Copy link
Collaborator Author

chalin commented Jul 29, 2021

Needs DCO before we can merge.

Sigh. Having only contributed to projects using CLA, I keep forgetting to sign. 🤷

@chalin chalin merged commit bda6e9b into main Jul 29, 2021
@chalin chalin deleted the chalin-local-build-preview-2021-07-29 branch July 29, 2021 17:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants