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

feat: convert to typescript (#76) #140

Merged
merged 6 commits into from
Feb 10, 2022
Merged

Conversation

achingbrain
Copy link
Member

@achingbrain achingbrain commented Jan 29, 2022

  • Converts to typescript
  • Only named exports
  • No more CJS, only ESM
  • Runs tests on all supported environments
  • Adds auto-publish
  • Adds dependabot

Fixes #139

BREAKING CHANGE: switch to named exports, ESM only

Adds gh actions, autorelease, etc.

BREAKING CHANGE: ESM only named exports no more CJS
src/index.ts Outdated
const { upgrader, filter } = opts

if (upgrader == null) {
throw new Error('An upgrader must be provided. See https://github.com/libp2p/interface-transport#upgrader.')
Copy link
Member

Choose a reason for hiding this comment

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

This and another reference to the interface-transport repo, should reference js-libp2p-interfaces instead

Copy link
Member

@wemeetagain wemeetagain left a comment

Choose a reason for hiding this comment

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

LGTM

@achingbrain achingbrain merged commit c4f6508 into master Feb 10, 2022
@achingbrain achingbrain deleted the feat/convert-to-typescript branch February 10, 2022 07:37
github-actions bot pushed a commit that referenced this pull request Feb 10, 2022
## [1.0.0](v0.16.2...v1.0.0) (2022-02-10)

### ⚠ BREAKING CHANGES

* switch to named exports, ESM only

### Features

* convert to typescript ([#76](#76)) ([#140](#140)) ([c4f6508](c4f6508)), closes [#139](#139)
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.

Potential memory leak
2 participants