-
Notifications
You must be signed in to change notification settings - Fork 21
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
Remove rpc-client crate #218
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the rust preflight lib need any of the rust.yml CI?
If no, looks good to me! 🎉
Thanks for following up the separation of the crate into the new repo. ❤️
Uhm, we already build it with RPC in all archs. So maybe I will just add a clippy test. |
ca82d6d
to
52ce357
Compare
e78adaa
to
7346a10
Compare
The PR is stuck due to the lack of @leighmcculloch Did you manage to find a workaround for https://github.com/orgs/community/discussions/12377 accross multiple workflows? I think that you used |
c340b97
to
236f19e
Compare
236f19e
to
d91b510
Compare
OK, I fixed it at d91b510 |
If you add a complete job to multiple workflows, as long as the workflows all start before one finishes, then the PR should be blocked from merging until all compelte jobs pass. At least htat's how it used to work It hink. |
I added a It works fine, but ... the problem is that if something fails and is re-run manually then you also need to re-run I may end up adding a |
The crate has moved to another repo: https://github.com/stellar/rs-stellar-rpc-client