Skip to content

Commit

Permalink
Merge pull request #2529 from aiidtest/fix-netlify-env-build
Browse files Browse the repository at this point in the history
Prevent netlify cli from pulling env variables
  • Loading branch information
kepae committed Jan 8, 2024
2 parents 5175118 + 825f57e commit b631943
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ jobs:
run: npm install netlify-cli -g

- name: Build using Netlify
run: netlify build --context deploy-preview
run: netlify build --context deploy-preview --offline
env:
NETLIFY_SITE_ID: ${{ vars.NETLIFY_SITE_ID }}
NETLIFY_AUTH_TOKEN: ${{ secrets.NETLIFY_AUTH_TOKEN }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ jobs:
run: npm install netlify-cli -g

- name: Build using Netlify
run: netlify build --context deploy-preview
run: netlify build --context deploy-preview --offline
working-directory: site/gatsby-site
env:
INSTRUMENT: true
Expand Down
5 changes: 4 additions & 1 deletion site/gatsby-site/github-netlify.toml
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,7 @@
# package = "/plugins/netlify-plugin-process-notifications"

[build.processing.html]
pretty_urls = false
pretty_urls = false

[context.deploy-preview]
publish = "public/"
5 changes: 4 additions & 1 deletion site/gatsby-site/tests-netlify.toml
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,7 @@
package = "@netlify/plugin-gatsby"

[build.processing.html]
pretty_urls = false
pretty_urls = false

[context.deploy-preview]
publish = "public/"

0 comments on commit b631943

Please sign in to comment.