bemuse icon indicating copy to clipboard operation
bemuse copied to clipboard

Refactoring: Replace `updeep` with `immer`

Open dtinth opened this issue 4 years ago • 6 comments

Background

Bemuse used the updeep library, but after some experience in other projects I think using immer leads to more readable code.

Task

  • Replace updeep with immer.
  • Your PR should migrate all usage of code that uses updeep and replace it with immer. In bemuse/package.json, updeep should be removed from dependencies, and latest version of immer added.
  • All automated checks should pass.

Hacktoberfest

If you would like to work on this task, please write a comment stating your intent. We will then assign the issue to you. To ensure continuity, the issue will be unassigned after 3 days of inactivity — so please keep us updated.

dtinth avatar Oct 17 '21 04:10 dtinth

Can I work on this issue?

aadityasinha-dotcom avatar Oct 17 '21 08:10 aadityasinha-dotcom

@aadityasinha-dotcom The issue is now yours, thank you! 🙏

dtinth avatar Oct 17 '21 08:10 dtinth

@dtinth I have added the latest version of immer in the bemuse/package.json and removed the updeep from the dependencies.

aadityasinha-dotcom avatar Oct 17 '21 10:10 aadityasinha-dotcom

@aadityasinha-dotcom I see that the PR is closed — do you plan to continue this some time or should I unassign this for now?

dtinth avatar Oct 20 '21 10:10 dtinth

@aadityasinha-dotcom I see that the PR is closed — do you plan to continue this some time or should I unassign this for now?

No, I am working on it. Actually that was an incomplete PR so I thought to close that.

aadityasinha-dotcom avatar Oct 20 '21 10:10 aadityasinha-dotcom

@aadityasinha-dotcom Thanks for the update! Let me know if you need any help. 😃

You can mark your PR as “draft” so that it is clear that the PR is not ready yet.

dtinth avatar Oct 20 '21 10:10 dtinth