-
Notifications
You must be signed in to change notification settings - Fork 8
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
fix: 🔧 Fixing env variable. #305
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
6f098c8
to
44f64dd
Compare
@SgtPooki So it looks like This test should've broken before merge, but it did not. SW seems to mess up playwright. I added a step to skip that This might have to be overridden, to merge so that this rule changes on Apologies for the inconvenience caused. 🙏🏽 |
@@ -2,9 +2,11 @@ name: CI | |||
|
|||
on: | |||
push: | |||
branches: [ "main" ] | |||
pull_request_target: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was the culprit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor
Fixes broken test: https://github.com/pln-planning-tools/Starmap/actions/runs/4119220968