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

Remove rpc-client crate #218

Merged
merged 6 commits into from
Jun 17, 2024
Merged

Conversation

2opremio
Copy link
Contributor

The crate has moved to another repo: https://github.com/stellar/rs-stellar-rpc-client

Copy link
Member

@leighmcculloch leighmcculloch left a 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. ❤️

@2opremio
Copy link
Contributor Author

Uhm, we already build it with RPC in all archs. So maybe I will just add a clippy test.

@2opremio 2opremio enabled auto-merge (squash) June 17, 2024 13:44
@2opremio
Copy link
Contributor Author

The PR is stuck due to the lack of complete job (marked as a required check in terraform).

@leighmcculloch Did you manage to find a workaround for https://github.com/orgs/community/discussions/12377 accross multiple workflows?

I think that you used complete for that but, as I understand it, it only works within a single workflow? I would like something similar across multiple workflows.

@2opremio 2opremio force-pushed the remove-rpc-client-crate branch 2 times, most recently from c340b97 to 236f19e Compare June 17, 2024 15:46
@2opremio 2opremio merged commit 647e64f into stellar:main Jun 17, 2024
19 checks passed
@2opremio 2opremio deleted the remove-rpc-client-crate branch June 17, 2024 16:17
@2opremio
Copy link
Contributor Author

OK, I fixed it at d91b510

@leighmcculloch
Copy link
Member

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.

@2opremio
Copy link
Contributor Author

2opremio commented Jun 18, 2024

I added a complete job in a new workflow (required_status_check.yml) which waits for the required workflows.

It works fine, but ... the problem is that if something fails and is re-run manually then you also need to re-run required_status_check, which is a pain.

I may end up adding a complete job to every required workflow, but that's also undersirable ...

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