Skip to content

Comments

Configure netlify via netlify.toml#551

Merged
wlach merged 2 commits intomainfrom
netlify-toml
Apr 26, 2021
Merged

Configure netlify via netlify.toml#551
wlach merged 2 commits intomainfrom
netlify-toml

Conversation

@wlach
Copy link
Contributor

@wlach wlach commented Apr 26, 2021

This is part of what we need to do to prepare for a production deploy (#537)

Pull Request checklist

  • The pull request has a descriptive title (and a reference to an issue it
    fixes, if applicable)
  • All tests and linter checks are passing
  • The pull request is free of merge conflicts

This is part of what we need to do to prepare for a production
deploy (#537).
We actually want storybook deploys for our staging server
(https://glean-dictionary-dev.netlify.app), as it may be useful
from time to time for debugging.
@wlach wlach requested a review from robotblake April 26, 2021 17:18
@wlach
Copy link
Contributor Author

wlach commented Apr 26, 2021

@robotblake You may want to unset the Build command and Publish directory settings in the netlify gui for dictionary.tmo, as the values in this file will now take precedence and it might be confusing to people in the future. Same with the PYTHON_VERSION environment variable.

command = """
scripts/build-glean-metadata.py
npm run build
if [ "$STORYBOOK" ]; then
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For glean-dictionary-dev (only), I've set this variable to 1 in the GUI. This seems like the most straightforward way of getting the right behaviour for our use case (multiple netlify "apps" for the same repo)

Copy link

@robotblake robotblake left a comment

Choose a reason for hiding this comment

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

I've gone ahead and removed the config from the Netlify Web UI. LGTM! 👍

@wlach wlach merged commit be5c896 into main Apr 26, 2021
@wlach wlach deleted the netlify-toml branch April 26, 2021 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants