Skip to content
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

(#195) Update .nvmrc file with correct Node Version #215

Merged

Conversation

seyilonge-ps
Copy link
Collaborator

@seyilonge-ps seyilonge-ps commented Aug 23, 2024

  • Updated .nvmrc and README.md files with correct Node version
  • Added engines property to package.json to specify Node and npm version ranges
  • Added .npmrc file and set engine-strict property to true

@seyilonge-ps seyilonge-ps force-pushed the ticket/195-update-files-with-correct-node-version branch 3 times, most recently from 145943b to 7b853da Compare August 23, 2024 18:12
@seyilonge-ps seyilonge-ps linked an issue Aug 26, 2024 that may be closed by this pull request
3 tasks
zoltan-dulac
zoltan-dulac previously approved these changes Aug 26, 2024
@zoltan-dulac
Copy link
Collaborator

@seyilonge-ps Is there any more work needed to be done here?

@seyilonge-ps
Copy link
Collaborator Author

seyilonge-ps commented Aug 26, 2024

@seyilonge-ps Is there any more work needed to be done here?

@zoltan-dulac Yes, the test script to check node versions specified in .nvmrc and README.md files match. Though that would be a cosmetic thing as the wrong version not specified in the engines property range would be rejected with the current update as is. Was also looking to increase that range a little bit, as it just differs by a minor version now, but that would need some extensive testing.

@seyilonge-ps seyilonge-ps force-pushed the ticket/195-update-files-with-correct-node-version branch from 7b853da to 1759ae7 Compare September 3, 2024 17:59
@seyilonge-ps seyilonge-ps marked this pull request as ready for review September 3, 2024 18:00
@seyilonge-ps seyilonge-ps force-pushed the ticket/195-update-files-with-correct-node-version branch 2 times, most recently from 4c06dfc to 71a3e02 Compare September 5, 2024 16:06
Closes #195

* Updated `.nvmrc` and `README.md` files with correct Node version
* Added `engines` property to `package.json` to specify Node and npm version ranges
* Added `.npmrc` file and set `engine-strict` property to `true`
* Updated Node version for GitHub actions
* Updated `README.md` file `Setup` section
* Added validation script file
* Removed `console.log` from test files
* Added `chromedriver_force_download=true` to `.npmrc` file
* Updated `chromedriver` from version `126.0.4` to `128.0.1`
* Removed `git add` task from `lint-staged` run script
* Updated chrome browser version for download in github `push-actions.yml` file
@seyilonge-ps seyilonge-ps force-pushed the ticket/195-update-files-with-correct-node-version branch from 71a3e02 to 63d10e2 Compare September 5, 2024 18:49
@zoltan-dulac zoltan-dulac merged commit d0a9384 into main Sep 9, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update .nvmrc file with correct Node Version
2 participants