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

Update version to 0.17.10-dev and tbtc.js to the latest range #405

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

Conversation

nkuba
Copy link
Member

@nkuba nkuba commented Sep 23, 2021

As @keep-network/tbtc.js version 0.19.4 was already published we need to bump up the range for pulling following versions in CI.

tbtc-dapp version 0.17.9 was already released so we need to start a new version in the main branch.

Refs #401

As @keep-network/tbtc.js version 0.19.4 was already published we need to
bump up the range for pulling following versions in CI.
Version 0.17.9 was already released so we need to start a new version.
@nkuba nkuba requested a review from a team September 23, 2021 13:15
"dependencies": {
"@0x/subproviders": "^6.0.8",
"@keep-network/tbtc.js": ">0.19.4-pre <0.19.4-rc",
"@keep-network/tbtc.js": ">0.19.5-pre <0.19.5-rc",
Copy link
Contributor

Choose a reason for hiding this comment

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

We no longer release the -pre-tagged packages.

Suggested change
"@keep-network/tbtc.js": ">0.19.5-pre <0.19.5-rc",
"@keep-network/tbtc.js": ">0.19.5-rc <0.19.5-ropsten",

And something that is outside of scope of this PR that I noticed when looking at the dependencies:
The dependencies in the package.json in @keep-network/tbtc.js don't refer to latest tbtc packages. I think we should use the "@keep-network/tbtc": ">1.1.2-dev <1.1.2-ropsten", dependency there, same as we use in coverage-pools. How do you think, @nkuba?

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