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

Testing out PR with build #656

Closed
wants to merge 3 commits into from

Conversation

plansmash-test
Copy link

Let's see if this works. I think PRs are broken do to secrets only in upstream repo

Copy link

netlify bot commented Nov 6, 2024

Deploy Preview for a11ycanada ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 516acba
🔍 Latest deploy log https://app.netlify.com/sites/a11ycanada/deploys/672cd62a0e829a0008113245
😎 Deploy Preview https://deploy-preview-656--a11ycanada.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

…d deploy

- Merged `update-analytics` and `deploy` jobs into a single consolidated job
- Updated workflow to run on schedule, manual dispatch, or after a PR is merged to `main`
- Simplified deployment logic to ensure analytics updates and deployment occur together post-merge
- Adjusted conditions to prevent deployment from running on every push, relying on Netlify for PR previews
@shawnthompson
Copy link
Contributor

PR was just a test from a non-maintainer account

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants