This repository was archived by the owner on Mar 5, 2025. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
Is semver being followed? 1.2.7 contains breaking changes #3758
Comments
Hey! Thanks for pointing out, ChainSafe recently took over the development of web3.js, our first release was |
LudwikJaniuk
added a commit
to LudwikJaniuk/web3.js
that referenced
this issue
Oct 27, 2020
see discussion in the issue web3#3758
Thank you, and it makes me happy to hear it's something you're taking seriously moving forward. Can I ask that you insert some note about it in the README? For example something like this: #3763, so that anyone coming across this in the future knows what's up. |
awesome thank you! |
spacesailor24
added a commit
that referenced
this issue
Mar 17, 2021
see discussion in the issue #3758 Co-authored-by: LudwikJaniuk <[email protected]>
spacesailor24
added a commit
that referenced
this issue
Mar 18, 2021
see discussion in the issue #3758 Co-authored-by: LudwikJaniuk <[email protected]> Co-authored-by: LudwikJaniuk <[email protected]>
GregTheGreek
pushed a commit
that referenced
this issue
Mar 23, 2021
* Rename tsc to compile (#3957) (#3958) * Change 'tsc' command in scripts & lerna run to 'compile' * Update changelog * Undo incorrect rename * Checking if e2e test is deterministic Co-authored-by: Sam Hodak <[email protected]> Co-authored-by: Sam Hodak <[email protected]> * Debugging failing tests (#3959) (#3960) * Remote node provider edits * remote node provider documentation edits Co-authored-by: smudgil <[email protected]> Co-authored-by: smudgil <[email protected]> * Clarify commitment to semantic versioning (#3961) (#3962) see discussion in the issue #3758 Co-authored-by: LudwikJaniuk <[email protected]> Co-authored-by: LudwikJaniuk <[email protected]> * Feature/web3 eth iban es6 (#3964) (#3965) * Feature/web3 eth iban es6 (#3964) * web3-eth-iban es6 rewrite - replace `var` with `const` and `let` * web3-eth-iban es6 rewrite - replace object prototype with class * web3-eth-iban code style fix Co-authored-by: Loredana <[email protected]> * Update changelog Co-authored-by: Loredana <[email protected]> * Update changelog for PR 3961 * Built lib Co-authored-by: Sam Hodak <[email protected]> Co-authored-by: smudgil <[email protected]> Co-authored-by: LudwikJaniuk <[email protected]> Co-authored-by: Loredana <[email protected]>
spacesailor24
added a commit
that referenced
this issue
Mar 26, 2021
* Replace hardcoded infura endpoints (#3943) * Replace hardcoded infura endpoints * infura link changelog * WARNING Exposing secrets, rotate infura keys after debugging is finished * Merging geth-dev-assistant updates (#3950) * Bump geth-dev-assistant to 0.1.9 * npm i * bump geth-dev-assistant * Remove logs * use posinstall to do deprecation warnings (#3917) * use posinstall to do deprecation warnings * add to change log * add preinstall warning * remove preinstall * add warnings to bzz and shh package.json Co-authored-by: Wyatt Barnes <[email protected]> * Bump elliptic from 6.5.3 to 6.5.4 in /packages/web3-eth-accounts (#3941) Bumps [elliptic](https://github.com/indutny/elliptic) from 6.5.3 to 6.5.4. - [Release notes](https://github.com/indutny/elliptic/releases) - [Commits](indutny/elliptic@v6.5.3...v6.5.4) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Wyatt Barnes <[email protected]> * Bump elliptic from 6.5.3 to 6.5.4 in /packages/web3-bzz (#3940) * Bump elliptic from 6.5.3 to 6.5.4 in /packages/web3-bzz Bumps [elliptic](https://github.com/indutny/elliptic) from 6.5.3 to 6.5.4. - [Release notes](https://github.com/indutny/elliptic/releases) - [Commits](indutny/elliptic@v6.5.3...v6.5.4) Signed-off-by: dependabot[bot] <[email protected]> * Update CHANGELOG Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Wyatt Barnes <[email protected]> Co-authored-by: Wyatt Barnes <[email protected]> * Bump elliptic from 6.5.3 to 6.5.4 in /packages/web3-core-requestmanager (#3945) * Bump elliptic from 6.5.3 to 6.5.4 in /packages/web3-core-requestmanager Bumps [elliptic](https://github.com/indutny/elliptic) from 6.5.3 to 6.5.4. - [Release notes](https://github.com/indutny/elliptic/releases) - [Commits](indutny/elliptic@v6.5.3...v6.5.4) Signed-off-by: dependabot[bot] <[email protected]> * Update CHANGELOG Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Wyatt Barnes <[email protected]> * add nvmrc file (#3817) * add nvmrc file * Update changelog Co-authored-by: Wyatt Barnes <[email protected]> Co-authored-by: Wyatt Barnes <[email protected]> * Rename tsc to compile (#3957) (#3958) * Change 'tsc' command in scripts & lerna run to 'compile' * Update changelog * Undo incorrect rename * Checking if e2e test is deterministic Co-authored-by: Sam Hodak <[email protected]> Co-authored-by: Sam Hodak <[email protected]> * Debugging failing tests (#3959) (#3960) * Remote node provider edits * remote node provider documentation edits Co-authored-by: smudgil <[email protected]> Co-authored-by: smudgil <[email protected]> * Clarify commitment to semantic versioning (#3961) (#3962) see discussion in the issue #3758 Co-authored-by: LudwikJaniuk <[email protected]> Co-authored-by: LudwikJaniuk <[email protected]> * Feature/web3 eth iban es6 (#3964) (#3965) * Feature/web3 eth iban es6 (#3964) * web3-eth-iban es6 rewrite - replace `var` with `const` and `let` * web3-eth-iban es6 rewrite - replace object prototype with class * web3-eth-iban code style fix Co-authored-by: Loredana <[email protected]> * Update changelog Co-authored-by: Loredana <[email protected]> Co-authored-by: Frankie <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Sam Hodak <[email protected]> Co-authored-by: smudgil <[email protected]> Co-authored-by: LudwikJaniuk <[email protected]> Co-authored-by: Loredana <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Hi, I my deployment scripts recently started crashing and I identified that a version change from Web3 1.2.1 to 1.2.9 was a possible issue. I'm not going to get into the details of that, it was just how I got here :)
I want to ask: the version numbers look like semantic versioning, so is that assumption correct of me to make? I couldn't quickly find any documentation on it here.
If you are using semantic versioning on this repository, then shouldn't version 1.2.7 have been version 2.0.0? The change notes state that behavior is being changed. Any external code that relied on the first user being used will fail after this. Given such a change, bumping only the patch number seems inappropriate to me. Am I missing something?
Best regards,
//Ludvig
The text was updated successfully, but these errors were encountered: