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

Clean up percy. - #2498

Merged
robdodson merged 1 commit into
masterfrom
task/percy-tweaks
Apr 3, 2020
Merged

Clean up percy.#2498
robdodson merged 1 commit into
masterfrom
task/percy-tweaks

Conversation

@robdodson

Copy link
Copy Markdown
Contributor

Changes proposed in this pull request:

  • Disable the glitch embed when percy is running against the components page.

I think at some point we should probably rename the site.env variable since it's specific to eleventy at this point but I didn't want to do that in this PR for fear of breaking something unseen.

@googlebot googlebot added the cla: yes Contributor has signed the CLA label Apr 2, 2020
@netlify

netlify Bot commented Apr 2, 2020

Copy link
Copy Markdown

Deploy preview for web-dev-staging ready!

Built with commit 915fb24

https://deploy-preview-2498--web-dev-staging.netlify.com

@MichaelSolati MichaelSolati left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Where is the environment set? Otherwise lgtm

@robdodson

Copy link
Copy Markdown
Contributor Author

@MichaelSolati it's set in package.json when you do npm run percy.

@robdodson
robdodson merged commit 5c164d6 into master Apr 3, 2020
@robdodson
robdodson deleted the task/percy-tweaks branch April 3, 2020 17:40
@MichaelSolati

Copy link
Copy Markdown
Contributor

Kk, I don't think I saw it in the changes, but kk

@robdodson

Copy link
Copy Markdown
Contributor Author

ah yeah it was done in a previous pr

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes Contributor has signed the CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants