Skip to content

Allows E2E to run from forks#2423

Merged
PeerRich merged 11 commits intomainfrom
e2e-fixes
Apr 10, 2022
Merged

Allows E2E to run from forks#2423
PeerRich merged 11 commits intomainfrom
e2e-fixes

Conversation

@zomars
Copy link
Copy Markdown
Contributor

@zomars zomars commented Apr 7, 2022

What does this PR do?

Sometimes running tests from forks fails due to missing variables. This allows us to run tests against our repo.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • Make a PR from a fork and tests should run normally

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 7, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

docs – ./apps/docs

🔍 Inspect: https://vercel.com/cal/docs/5p25aMubvQ3hJAoBzKK3cswVYWWb
✅ Preview: https://docs-git-e2e-fixes-cal.vercel.app

[Deployment for bc7cbdc canceled]

calendso – ./apps/web

🔍 Inspect: https://vercel.com/cal/calendso/4LwyPJp15hHAFEx9sE7ykcxDz2Q9
✅ Preview: https://calendso-git-e2e-fixes-cal.vercel.app

@zomars zomars requested review from a team and alannnc April 7, 2022 18:25
@zomars zomars added ♻️ autoupdate tells kodiak to keep this branch up-to-date automerge labels Apr 7, 2022
@vercel vercel Bot temporarily deployed to Preview – docs April 7, 2022 18:25 Inactive
@vercel vercel Bot temporarily deployed to Preview – calendso April 7, 2022 18:28 Inactive
@agustif
Copy link
Copy Markdown
Contributor

agustif commented Apr 7, 2022

LGTM, Seems to be running them, will wait for output. #2427

Copy link
Copy Markdown
Contributor

@agustif agustif left a comment

Choose a reason for hiding this comment

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

Awesome fix for external contributors to get input on test suite without our pre-approval.
LGTM!

@vercel vercel Bot temporarily deployed to Preview – calendso April 8, 2022 05:33 Inactive
@vercel vercel Bot temporarily deployed to Preview – calendso April 8, 2022 11:52 Inactive
@vercel vercel Bot temporarily deployed to Preview – docs April 8, 2022 11:52 Inactive
@vercel vercel Bot temporarily deployed to Preview – calendso April 8, 2022 16:50 Inactive
@vercel vercel Bot temporarily deployed to Preview – docs April 8, 2022 16:50 Inactive
@vercel vercel Bot temporarily deployed to Preview – calendso April 8, 2022 16:59 Inactive
@vercel vercel Bot temporarily deployed to Preview – docs April 9, 2022 14:40 Inactive
@vercel vercel Bot temporarily deployed to Preview – calendso April 9, 2022 14:40 Inactive
@vercel vercel Bot temporarily deployed to Preview – docs April 9, 2022 22:33 Inactive
@vercel vercel Bot temporarily deployed to Preview – calendso April 9, 2022 22:33 Inactive
@vercel vercel Bot temporarily deployed to Preview – docs April 10, 2022 15:57 Inactive
@vercel vercel Bot temporarily deployed to Preview – calendso April 10, 2022 15:57 Inactive
@PeerRich PeerRich merged commit b6da0f0 into main Apr 10, 2022
@PeerRich PeerRich deleted the e2e-fixes branch April 10, 2022 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants