test: remove example-install-only#1838
Open
MikeMcC399 wants to merge 1 commit into
Open
Conversation
Collaborator
|
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
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.
Situation
The workflow example-install-only.yml pins a version of Cypress separately to the version defined in examples/install-only.
Renovate is unable to maintain this version, resulting in manual instructions being written for
Code scanning typically flags pinning npm versions in workflows like this, since current updating utilities such as Renovate and Dependabot can't maintain them.
The workflow currently runs on ubuntu-24.04 using the default Node.js 22.23.1 version.
If it is upgraded to Node.js 24.x, it warns. And if it is upgraded to npm 12, it fails.
Attempting to make it compatible with npm 12, exposes other errors in the workflow, such as the npm install running in the wrong directory.
Assessment
Given heightened awareness of supply-chain vulnerabilities, it can no longer be considered as a good example and it is not fixable. npm does not allow picking only one dependency defined in
package.json, although it can choose betweendevandprod. Renovate can't keep it synced either.The higher frequency of reported vulnerabilities in the npm ecosystem, compared to earlier, means that it is no longer viable to be manually updating dependencies. The main part of the work needs to be automated, and in this repo that means relying on Renovate.
Change
Remove the following, and all references to these examples:
Verification
Under Ubuntu 24.04.4 LTS, Node.js 24.18.0 LTS execute:
Confirm that the script runs without error and no longer prompts to manually update example-install-only.yml.
Note
Low Risk
Documentation and example removal only; no runtime changes to the GitHub Action itself.
Overview
Removes the install-only pattern from the repo: the
example-install-onlyGitHub Actions workflow, theexamples/install-onlyproject, and all documentation that described installing only Cypress (withinstall: falseon the action) instead of fullnpm ci.README drops the table-of-contents link and the full “Install Cypress only” section (sample YAML and status badge). Custom install now ends after the note that you may install and cache Cypress yourself without the action—without pointing at this example.
Maintenance is simplified:
docs/MAINTENANCE.mdno longer mentions hand-editing the workflow’s pinned Cypress version;scripts/update-cypress-latest-npm.shno longer updatesinstall-only; andscripts/update-cypress-latest.shno longer prints/opens instructions to manually sync that workflow.Reviewed by Cursor Bugbot for commit 67074b8. Bugbot is set up for automated code reviews on this repo. Configure here.