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

chore: update the package name and version #7

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

nazarhussain
Copy link

Motivation

Update the package name and version for standalone release as @chainsafe/benchmark.

Description

  • Update the package version
  • Update the package name
  • Update docs

Steps to test or reproduce

  • Run all tests

Copy link
Member

@matthewkeil matthewkeil left a comment

Choose a reason for hiding this comment

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

Couple small changes and the question about leaving dapplion's name in the comments to pay homage to his work on this before we forked it

README.md Outdated Show resolved Hide resolved
package.json Outdated
"repository": "[email protected]:dapplion/benchmark.git",
"author": "dapplion <[email protected]>",
"name": "@chainsafe/benchmark",
"version": "0.1.0.rc-0",
Copy link
Member

@matthewkeil matthewkeil Dec 9, 2024

Choose a reason for hiding this comment

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

Suggested change
"version": "0.1.0.rc-0",
"version": "1.1.0",

Copy link
Author

Choose a reason for hiding this comment

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

As it's a new package I would prefer to start with 0 versioning scheme.

https://0ver.org/

Copy link
Member

Choose a reason for hiding this comment

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

its not a new package though and it was forked from 1.0.0.... if anything it should be 1.1.0 if this is a minor and 2.0.0 if its a major which i dont think it is....

1.1.0 feels like the right number here

Copy link
Member

Choose a reason for hiding this comment

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

and no RC

src/utils/gaContext.ts Outdated Show resolved Hide resolved
Copy link
Member

@matthewkeil matthewkeil left a comment

Choose a reason for hiding this comment

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

LGTM 🎸

Base automatically changed from nh/embeded-runner to main December 12, 2024 16:03
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