responsively-app icon indicating copy to clipboard operation
responsively-app copied to clipboard
verified publisher icon responsively-org

Chore/upgrade dependencies

Open rabrowne85 opened this issue 5 months ago â€ĸ 5 comments

✨ Pull Request

📓 Referenced Issue

There is no specific issue that directly prompted this PR, but it includes a fix for:

  • Fixes #1193

ℹī¸ About the PR

This PR updates all project dependencies to their latest compatible versions, as resolved by the package manager.

Key changes include:

  • ElectronJS has been upgraded to a more recent release, which includes a newer Chromium version. This enables support for modern web APIs and features.
  • TailwindCSS has been upgraded to v4.

While no breaking changes have been identified so far, due to the broad scope of dependency updates, this might be best classified as a minor version bump. Please advise if a major version is more appropriate.

Note: The project does not appear to have a comprehensive test suite, though npm run test completes successfully. 🤷‍♂ī¸

đŸ–ŧī¸ Testing Scenarios / Screenshots

The following commands were run to validate the changes:

  • npm run start
  • npm run build
  • npm run package
  • npm run test

While the test script completes, it's unclear how much coverage it provides.

Screenshots to follow

Items Outstanding

  • [x] Preload script cannot be loaded into the webviews - therefore browserSync does not work.
  • [x] React Dev Tools does not load.

rabrowne85 avatar Jul 17 '25 20:07 rabrowne85

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jul 17 '25 20:07 CLAassistant

Any update on this? @rabrowne85 @manojVivek

radiorambo avatar Nov 13 '25 08:11 radiorambo

Any update on this? @rabrowne85 @manojVivek

No real update - there is one outstanding issue that I've had on the back burner which is keeping the views in sync with each other. For some reason the browser sync package is no longer keeping things together and there's no obvious reason why not. I'll try and get a couple of hours on it this morning 👍

rabrowne85 avatar Nov 13 '25 09:11 rabrowne85

@manojVivek I've managed to get the preload to work and as such the browserSync also works.

I see that the test workflow fails as it's using node 16 - any objections bumping this to at least 18?

I've merged the current master branch in so that any recent updates are included.

Think the main "breaking changes" are going to be from the electron update, i.e., macOS 11 or older are no longer supported. Do you want to go to v2.x or v1.18.x?

rabrowne85 avatar Nov 13 '25 14:11 rabrowne85

@manojVivek Any thoughts on the questions of whether to go v2 and/or adjust the config of the GitHub action? Would be good to get this in for review and merged 👍

rabrowne85 avatar Nov 23 '25 10:11 rabrowne85