Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replaced @bufbuild/connect deprecated dependency with @connectrpc #7

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

chamorin
Copy link
Contributor

@chamorin chamorin commented Nov 16, 2023

  • Replaced @bufbuild/connect deprecated dependency with @connectrpc/connect and @connectrpc/connect-node
  • Upgraded @substreams/core version to latest
  • Removed progress state tracking because it seems like it's not supported anymore by @substreams/core

PR-Codex overview

Focus of the PR:

Update dependencies and import statements.

Detailed summary:

  • Updated the version of @substreams/node in package.json to "0.4.0".
  • Updated the version of @substreams/node in package-lock.json to "0.4.0".
  • Updated the version of @substreams/core in package.json to "^0.7.0".
  • Updated the version of @bufbuild/protobuf in package.json to "latest".
  • Updated the version of @connectrpc/connect in package.json to "latest".
  • Updated the version of @connectrpc/connect-node in package.json to "latest".
  • Updated the version of eventemitter3 in package.json to "latest".
  • Updated the version of @bufbuild/protobuf in package-lock.json to "latest".
  • Updated the version of @connectrpc/connect in package-lock.json to "latest".
  • Updated the version of @connectrpc/connect-node in package-lock.json to "latest".
  • Updated the version of eventemitter3 in package-lock.json to "latest".
  • Updated the import statements in src/createHeadersInterceptor.ts to use @connectrpc/connect instead of @bufbuild/connect.
  • Updated the import statements in src/createDefaultTransport.ts to use @connectrpc/connect and @connectrpc/connect-node instead of @bufbuild/connect and @bufbuild/connect-node.
  • Updated the import statements in src/BlockEmitter.ts to use @connectrpc/connect instead of @bufbuild/connect.
  • Removed the import statement for createStateTracker in src/BlockEmitter.ts.
  • Updated the event types in src/BlockEmitter.ts to remove the state argument from some events.

Note: Imports and changes in package.json and package-lock.json are not mentioned as per the instructions.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

…nect and @connectrpc/connect-node and removed progress state
Copy link

changeset-bot bot commented Nov 16, 2023

⚠️ No Changeset found

Latest commit: dca7478

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

commit-lint bot commented Nov 16, 2023

Contributors

chamorin

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

  • @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
  • @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
  • @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
  • @Commit-Lint merge disable will desactivate merge dependabot PR
  • @Commit-Lint review will approve dependabot PR
  • @Commit-Lint stop review will stop approve dependabot PR

@chamorin chamorin changed the title Replaced @bufbuild/connect deprecated dependency @connectrpc Replaced @bufbuild/connect deprecated dependency with @connectrpc Nov 16, 2023
Copy link
Contributor

@DenisCarriere DenisCarriere left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@DenisCarriere DenisCarriere merged commit b5063de into substreams-js:main Nov 17, 2023
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants