Skip to content
This repository has been archived by the owner on Feb 19, 2020. It is now read-only.

feature(streamer): POC of streamer layer and project reorg #8

Merged
merged 2 commits into from
Nov 1, 2018

Conversation

AlexCppns
Copy link
Contributor

In this PR:

  • First migration of StreamHandler class
  • Split directory in src and dist
  • Add @dfuse namespace in package name.

@AlexCppns AlexCppns requested a review from maoueh October 31, 2018 18:18
Copy link
Contributor

@maoueh maoueh left a comment

Choose a reason for hiding this comment

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

Apart from the publish thing, rest look good for now.

@@ -6,9 +6,9 @@
"types": "index.d.js",
"files": [
"index.js",
"index.d.ts",
"types"
Copy link
Contributor

Choose a reason for hiding this comment

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

Removing this will cause the publish to not include them in the package, which is not what is expected.

package.json Outdated
],
"bin": "dist",
Copy link
Contributor

Choose a reason for hiding this comment

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

In fact, I see you changed the source files and their output directory. Probably something related to that caused the bad publish.

Will need to check that before merging.

@maoueh maoueh merged commit 3d2b5da into master Nov 1, 2018
@maoueh maoueh deleted the eosws-migration branch November 1, 2018 15:29
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.

2 participants