test(e2e): remove e2e job and verdaccio setup after CRA/webpack template removal#1524
Merged
Merged
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1524 +/- ##
=======================================
Coverage 93.58% 93.58%
=======================================
Files 5 5
Lines 78 78
Branches 13 13
=======================================
Hits 73 73
Misses 4 4
Partials 1 1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
…ate removal Remove `scripts/e2e.ts`, `scripts/verdaccio.yaml`, and `scripts/npmrc` as all CRA templates, webpack template, and custom react-scripts have been removed from the monorepo. Remove `verdaccio` dev dependency, clean up `.gitignore` entries, and remove the e2e CI job from publish/deploy needs. Also remove stale CRA-related keywords from `package.json`.
sabertazimi
force-pushed
the
chore/remove-e2e-infrastructure
branch
from
June 5, 2026 09:42
4c8586e to
2f0879f
Compare
sabertazimi
changed the title
github-actions Bot
added a commit
that referenced
this pull request
Jun 5, 2026
…ate removal (#1524) Remove `scripts/e2e.ts`, `scripts/verdaccio.yaml`, and `scripts/npmrc` as all CRA templates, webpack template, and custom react-scripts have been removed from the monorepo. Remove `verdaccio` dev dependency, clean up `.gitignore` entries, and remove the e2e CI job from publish/deploy needs. Also remove stale CRA-related keywords from `package.json`. 870b7b6
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
CRA templates, webpack template, and custom react-scripts (
@sabertazimi/react-scripts) have already been removed from the monorepo. The e2e testing infrastructure that tested those templates is now entirely obsolete.Changes
Deleted files
scripts/e2e.ts— E2E test harness for CRA/webpack templatesscripts/verdaccio.yaml— Local npm registry config (only used by e2e)scripts/npmrc— localhost registry pointer (only used by CI e2e job)Modified files
package.json— RemoveverdacciodevDependency (-151 packages); remove stale keywords (create-react-app,react,react-scripts).github/workflows/ci.yml— Remove entiree2eCI job; removee2efrompublishanddeployjobneeds.gitignore— Remove verdaccio-related entries (scripts/htpasswd,scripts/storage/,storage/,.npmrc)AGENTS.md— Update build scripts documentationTest plan
pnpm lintpassespnpm installsucceeds (verdaccio and 151 transitive packages removed)