Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Deployment previews for pull requests #799

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

jasonbarry
Copy link

I think Truffle Suite would benefit in having deployment previews on every pull request, that way we can generate an automatic build of the website per PR rather than having to checkout branches locally to review/verify changes.

These are the changes needed to integrate w/ FeaturePeek, a deployment preview provider. If the GitHub App is installed (for webhooks), then every subsequent PR will have its own deployment preview.

@jasonbarry jasonbarry changed the title Integrate with FeaturePeek Deployment previews for pull requests Oct 1, 2020
@featurepeek-bot
Copy link

This pull request has been automatically deployed to FeaturePeek. 🔥
Your deployment will be kept up-to-date with this pull request's latest changes.

https://peek.run/5v3hde5v


Please read our docs for more configuration details.

@OnlyOneJMJQ
Copy link
Contributor

Hi @JasonBerry, this looks really cool! Nice guerrilla marketing idea. We'll give it a spin and let you know how it goes! To your point, I just spent more time than I'd like pulling in a PR, spinning up a dev server, etc. just to tests responsiveness.

Before merging, do we need to create a FeaturePeek account?

@jasonbarry
Copy link
Author

@DiscRiskandBisque you can merge either before or after creating a FeaturePeek account. Since you're a member of the Truffle Suite org, when you create a team you'll be assigned as the team owner. It's free to use for all open source repos. Hope you enjoy!

main:
type: static
path: build
spa: false
Copy link
Contributor

Choose a reason for hiding this comment

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

Needs newline at end of file

@cds-amal cds-amal assigned kevinbluer and unassigned cds-amal Jun 2, 2021
@cds-amal cds-amal requested review from kevinbluer and removed request for cds-amal June 2, 2021 18:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants