BioDrop icon indicating copy to clipboard operation
BioDrop copied to clipboard

Use fs promises

Open swiftugandan opened this issue 2 years ago • 1 comments

Fixes Issue

Closes #2815 and #2640

Changes proposed

  • Uses fs.promises for filesystem calls
  • Adds an lru-cache to reduce the number of filesystem calls

Check List (Check all the applicable boxes)

  • [x] My code follows the code style of this project.
  • [ ] My change requires changes to the documentation.
  • [ ] I have updated the documentation accordingly.
  • [x] All new and existing tests passed.
  • [x] This PR does not contain plagiarized content.
  • [x] The title of my pull request is a short description of the requested changes.

swiftugandan avatar Jan 10 '23 22:01 swiftugandan

Welcome @swiftugandan! Thank you so much for your first pull request!

github-actions[bot] avatar Jan 10 '23 22:01 github-actions[bot]

@swiftugandan thank you for your contribution and efforts with this, but I am going to close this pull request as I think it needs more thought and maybe in small increment steps

eddiejaoude avatar Jan 14 '23 12:01 eddiejaoude

@swiftugandan thank you for your contribution and efforts with this, but I am going to close this pull request as I think it needs more thought and maybe in small increment steps

Ok

swiftugandan avatar Jan 14 '23 12:01 swiftugandan