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

refactor: build synchronizer using webpack #486

Closed
wants to merge 6 commits into from

Conversation

devcatalin
Copy link
Member

This PR...

Changes

  • Had to use webpack v4 because v5 wouldn't build, giving me only errors
  • Downgraded a bunch of dependencies to earlier versions so that they can work with webpack 4

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

@changeset-bot
Copy link

changeset-bot bot commented Aug 22, 2023

🦋 Changeset detected

Latest commit: eca9e2f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@monokle/synchronizer Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@devcatalin devcatalin requested a review from f1ames August 22, 2023 11:55
@f1ames
Copy link
Contributor

f1ames commented Aug 23, 2023

I'm digging into this now, still couple of issues:

@devcatalin
Copy link
Member Author

Closed this PR in favor of #488

@devcatalin devcatalin closed this Aug 23, 2023
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.

2 participants