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

Bump up versions of the dependencies. #269

Closed
wants to merge 2 commits into from
Closed

Bump up versions of the dependencies. #269

wants to merge 2 commits into from

Conversation

silvestrpredko
Copy link

No description provided.

@silvestrpredko
Copy link
Author

Unfortunately, seems like my PR brokes everything.

@ChaoticTempest
Copy link
Member

@silvestrpredko It looks like you deleted the workspaces types which was hiding CryptoHash before. It's not the same interface so things likely broke internally. We're waiting on near_primitives to be in 1.0 version before we reliably use it directly

@silvestrpredko
Copy link
Author

Yeah seems like you are right. By the way, I am working on a library client for Near. https://github.com/Relayz-io/near-client It's basically an implementation of a NearRPC. And I am dependent on near-primitives. It would be great to see the stable version soon.

@ChaoticTempest
Copy link
Member

I see, yeah let's just bump up the versions without deleting that workspaces types then. There's no timeline for the stable version of near-primitives. 0.16 version shouldn't conflict besides needing to bump the Minimum support rust version to 1.67.1

	```near-account-id```, ```near-crypto```,
	```near-primitives```, ```near-jsonrpc-primitives```,
	```near-jsonrpc-client```.
@silvestrpredko
Copy link
Author

@ChaoticTempest Done ✅. Please check it out.

@ChaoticTempest
Copy link
Member

closing this in favor of #271 since the tests are passing for those

@silvestrpredko silvestrpredko deleted the silvestr/bummp-up-dependencies-version branch May 26, 2023 23:20
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