Skip to content
This repository has been archived by the owner on May 5, 2023. It is now read-only.

Version 5 refactor #31

Merged
merged 12 commits into from
Apr 26, 2023
Merged

Version 5 refactor #31

merged 12 commits into from
Apr 26, 2023

Conversation

TooTallNate
Copy link
Owner

Changes include:

Major, breaking changes

Minor changes

  • Proper options typings are now inferred based on the protocol of the URI provided to getUri(). For example: getUri('ftp://...') will have FtpOptions as the second parameter implicitly.
  • A URL instance can now be provided as the first argument to getUri().

Misc

  • Use pnpm for GH Actions.
  • Use Jest for tests.
  • Refactor tests to TypeScript.

@TooTallNate TooTallNate merged commit 0b7b999 into master Apr 26, 2023
@TooTallNate TooTallNate deleted the v5 branch April 26, 2023 04:24
@jimmywarting
Copy link

While making such a huge refactor, would it be good to switch to ESM then also?
it still compiles down to cjs...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dependency on 'ftp' causes [DEP0005] DeprecationWarning: Buffer() switch ftp client?
2 participants