Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add pr testing for component explorer #5365

Merged
merged 4 commits into from
Nov 6, 2021

Conversation

awentzel
Copy link
Collaborator

@awentzel awentzel commented Nov 4, 2021

Pull Request

📖 Description

Implements PR testing for the component explorer site.

  • Deploys to Azure Static Web Apps
  • Generates link to view the site embedded in PR comment
  • Redefines workflow with jobs running in parallel
  • Improves perf sharing between jobs with uploading/downloading artifacts so only have to build once

🎫 Issues

n/a, feature addition

👩‍💻 Reviewer Notes

This will require testing several key scenarios.

  1. only merges to master branch should fire documentation build and deployment to https://stage.www.fast.design
  2. only PR events should fire build and deployment to static web apps and that content will be destroyed on pr close.
  3. Verify outside contributors
  4. Verify inside contributors
  5. Verify on merge to master

📑 Test Plan

✅ Checklist

General

  • I have included a change request file using $ yarn change
  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

⏭ Next Steps

Replicate this workflow change to other deployment sites.

@awentzel awentzel added area:dev-ops Pertains to build, CI, and other dev-ops work area:cloud labels Nov 4, 2021
@awentzel awentzel self-assigned this Nov 4, 2021
@awentzel awentzel changed the title add component explorer pr testing add pr testing for component explorer Nov 4, 2021
@github-actions
Copy link

github-actions bot commented Nov 4, 2021

Azure Static Web Apps: Your stage site is ready! Visit it here: https://blue-coast-0df7a6610-5365.centralus.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Nov 6, 2021

Azure Static Web Apps: Your stage site is ready! Visit it here: https://blue-coast-0df7a6610-5365.centralus.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Nov 6, 2021

Azure Static Web Apps: Your stage site is ready! Visit it here: https://purple-ocean-0b7ce3410-5365.centralus.azurestaticapps.net

@awentzel awentzel merged commit b122988 into master Nov 6, 2021
@awentzel awentzel deleted the users/awentzel/add-stapp-explorer branch November 6, 2021 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dev-ops Pertains to build, CI, and other dev-ops work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants