Skip to content

devops: run tests using Docker image#1788

Merged
mxschmitt merged 1 commit intomainfrom
use-gha-docker-container
Jun 26, 2025
Merged

devops: run tests using Docker image#1788
mxschmitt merged 1 commit intomainfrom
use-gha-docker-container

Conversation

@mxschmitt
Copy link
Contributor

@mxschmitt mxschmitt commented Jun 26, 2025

Motivation: 4:50min -> 3:6min

@github-actions
Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-forest-0a29f6210-1788.centralus.azurestaticapps.net

@mxschmitt mxschmitt force-pushed the use-gha-docker-container branch from dc09edc to a44f752 Compare June 26, 2025 10:21
@github-actions
Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-forest-0a29f6210-1788.centralus.azurestaticapps.net

@mxschmitt mxschmitt requested a review from Skn0tt June 26, 2025 10:33
@mxschmitt mxschmitt marked this pull request as ready for review June 26, 2025 10:33
uses: Azure/static-web-apps-deploy@v1
with:
azure_static_web_apps_api_token: ${{ secrets.AZURE_STATIC_WEB_APPS_API_TOKEN_DELIGHTFUL_FOREST_0A29F6210 }}
app_location: "/"
Copy link
Member

Choose a reason for hiding this comment

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

this seems unrelated, could you elaborate?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

VSCode was giving me a warning since its a required field.

Copy link
Member

Choose a reason for hiding this comment

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

aha. sounds like a very required field if it worked before ^^

@mxschmitt mxschmitt merged commit 70a5b6f into main Jun 26, 2025
5 checks passed
@mxschmitt mxschmitt deleted the use-gha-docker-container branch June 26, 2025 12:12
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.

2 participants