Skip to content

Fix CI dependencies - #475

Merged
MrGreenTea merged 5 commits into
mainfrom
MrGreenTea-patch-1
Nov 13, 2024
Merged

Fix CI dependencies#475
MrGreenTea merged 5 commits into
mainfrom
MrGreenTea-patch-1

Conversation

@MrGreenTea

@MrGreenTea MrGreenTea commented Nov 12, 2024

Copy link
Copy Markdown
Contributor

#450 was merged automatically before I could push this.

Without this the lint stage won't run if the playwright tests fail because it waits for all other jobs to finish.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 12, 2024

Copy link
Copy Markdown

Deploying website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1e7c73e
Status: ✅  Deploy successful!
Preview URL: https://929e9c59.website-7c4.pages.dev
Branch Preview URL: https://mrgreentea-patch-1.website-7c4.pages.dev

View logs

#450 was merged automatically before I could push this
@MrGreenTea

Copy link
Copy Markdown
Contributor Author

I accidentally pushed to main with 0944b63
Whoopsie

Because prettier outputs a summary line with [warn] this might cause some "file not found" issues though.
@MrGreenTea
MrGreenTea marked this pull request as draft November 12, 2024 15:31
@MrGreenTea

Copy link
Copy Markdown
Contributor Author

So don't skip it. This will just mask mistakes.
@MrGreenTea
MrGreenTea marked this pull request as ready for review November 13, 2024 11:48
@MrGreenTea
MrGreenTea merged commit 3002380 into main Nov 13, 2024
@MrGreenTea
MrGreenTea deleted the MrGreenTea-patch-1 branch November 13, 2024 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant