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

Remove browser-sync #27772

Merged
merged 1 commit into from
Feb 12, 2025
Merged

Remove browser-sync #27772

merged 1 commit into from
Feb 12, 2025

Conversation

jamesmockett
Copy link
Contributor

What does this change?

Removes browser-sync package and updates watch task so that it no longer relies upon it.

Note

The removal of browser-sync means that hot reloading is no longer supported in frontend. The watch task will continue to watch for updates to Sass files, recompile them, and notify you when this has been done, but you will need to manually refresh your browser to see any changes.

@jamesmockett jamesmockett requested a review from a team as a code owner February 11, 2025 17:15
@jamesmockett jamesmockett self-assigned this Feb 11, 2025
@jamesmockett jamesmockett added the dependencies Pull requests that update a dependency file label Feb 11, 2025
@jamesmockett jamesmockett added this to the Health milestone Feb 11, 2025
Copy link
Contributor

@DanielCliftonGuardian DanielCliftonGuardian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

Co-authored-by: Daniel Clifton <[email protected]>
Co-authored-by: Jamie B <[email protected]>
Co-authored-by: Marjan Kalanaki <[email protected]>
Co-authored-by: Simon Adcock <[email protected]>
@jamesmockett jamesmockett force-pushed the mob/remove-browser-sync branch from f0d365f to dd183be Compare February 12, 2025 10:58
@jamesmockett jamesmockett merged commit b3dbe21 into main Feb 12, 2025
6 checks passed
@jamesmockett jamesmockett deleted the mob/remove-browser-sync branch February 12, 2025 14:18
@prout-bot
Copy link
Collaborator

Seen on FRONTS-PROD, ADMIN-PROD (merged by @jamesmockett 12 minutes and 32 seconds ago)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Seen-on-ADMIN-PROD Seen-on-FRONTS-PROD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants