[PR #8067] [MERGED] ♻️ remove unnecessary playwright container pulls to speed up CI #114353

Closed
opened 2026-06-05 22:11:44 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/8067
Author: @matt-fidd
Created: 6/3/2026
Status: Merged
Merged: 6/3/2026
Merged by: @matt-fidd

Base: masterHead: ci/playwright-speedup


📝 Commits (2)

📊 Changes

3 files changed (+6 additions, -14 deletions)

View changed files

📝 .github/workflows/e2e-test.yml (+0 -6)
📝 .github/workflows/vrt-update-generate.yml (+0 -8)
upcoming-release-notes/8067.md (+6 -0)

📄 Description

Description

I was looking at ways to speed our CI up/use less resources. I think these are left over from when all of our jobs were consolidated and are no longer needed, as far as I can tell.

Testing

I haven't, but I've verified the workflows I've removed it from don't appear to need them, and this is pretty risk free to revert if it breaks anything.

Checklist

  • Release notes added (see link above)
  • No obvious regressions in affected areas
  • Self-review has been performed - I understand what each change in the code does and why it is needed

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/8067 **Author:** [@matt-fidd](https://github.com/matt-fidd) **Created:** 6/3/2026 **Status:** ✅ Merged **Merged:** 6/3/2026 **Merged by:** [@matt-fidd](https://github.com/matt-fidd) **Base:** `master` ← **Head:** `ci/playwright-speedup` --- ### 📝 Commits (2) - [`bc23116`](https://github.com/actualbudget/actual/commit/bc231160c2fc76f94cf175a90d6bd4ece48676f2) don't use playwright where it's not needed - [`63a2553`](https://github.com/actualbudget/actual/commit/63a2553b9eaab62562f4a91aa0eba7164e9fea6f) note ### 📊 Changes **3 files changed** (+6 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/e2e-test.yml` (+0 -6) 📝 `.github/workflows/vrt-update-generate.yml` (+0 -8) ➕ `upcoming-release-notes/8067.md` (+6 -0) </details> ### 📄 Description <!-- Thank you for submitting a pull request! Make sure to follow the instructions to write release notes for your PR — it should only take a minute or two: https://actualbudget.org/docs/contributing/#writing-good-release-notes. Try running yarn generate:release-notes *before* pushing your PR for an interactive experience. --> ## Description <!-- What does this PR do? Why is it needed? Please give context on the "why?": why do we need this change? What problem is it solving for you?--> I was looking at ways to speed our CI up/use less resources. I think these are left over from when all of our jobs were consolidated and are no longer needed, as far as I can tell. ## Related issue(s) <!-- e.g. Fixes #123, Relates to #456 --> ## Testing <!-- What did you test? How can we reproduce the issue you are fixing or how can we test the feature you built? --> I haven't, but I've verified the workflows I've removed it from don't appear to need them, and this is pretty risk free to revert if it breaks anything. ## Checklist - [x] Release notes added (see link above) - [x] No obvious regressions in affected areas - [x] Self-review has been performed - I understand what each change in the code does and why it is needed <!--- actual-bot-sections ---> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-06-05 22:11:44 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#114353