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

Azure SWA Staging Site(s) #334

Merged
merged 14 commits into from
Aug 10, 2024
Merged

Azure SWA Staging Site(s) #334

merged 14 commits into from
Aug 10, 2024

Conversation

PipeItToDevNull
Copy link
Member

@PipeItToDevNull PipeItToDevNull commented Aug 9, 2024

This PR adds a new GHA that will build "master" and up to 3? PRs against master into Azure SWA's and serve them at https://staging.rtech.support/ + 3 other random domains, which are a bit harder to find but are in the GHA output.

Implementation of this requires an edit to Gemfile which provides a version of Ruby to use. I am declaring ruby '>=3.0' because this is required by nokogiri

This could break people on MacOS which ships with 2.6.7, you need to use this fix on MacOS because the latest version of Ruby logger is also broken.

You may need to bundle update as well if you can start the serve command but it errors out after trying to start up.

@PipeItToDevNull PipeItToDevNull marked this pull request as ready for review August 9, 2024 03:21
Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-mushroom-0564bdf0f-334.eastus2.5.azurestaticapps.net

Copy link
Contributor

@zleyyij zleyyij left a comment

Choose a reason for hiding this comment

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

This looks really fucking cool, good shit

@PipeItToDevNull PipeItToDevNull merged commit cd584ee into master Aug 10, 2024
2 checks passed
@PipeItToDevNull PipeItToDevNull deleted the azure-swa-test branch August 10, 2024 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants