Skip to content

chore: Remove one level of nesting in remote prover client modules#1626

Merged
Mirko-von-Leipzig merged 3 commits intonextfrom
sergerad-mod-nesting
Feb 2, 2026
Merged

chore: Remove one level of nesting in remote prover client modules#1626
Mirko-von-Leipzig merged 3 commits intonextfrom
sergerad-mod-nesting

Conversation

@sergerad
Copy link
Collaborator

@sergerad sergerad commented Feb 1, 2026

As per comment.

the namespacing here feels a bit "too nested" - e.g., could this not be just miden_remote_prover_client::block_prover::RemoteBlockProver?

@sergerad sergerad requested a review from bobbinth February 1, 2026 21:55
@sergerad sergerad added the no changelog This PR does not require an entry in the `CHANGELOG.md` file label Feb 1, 2026
use miden_protocol::block::{BlockHeader, BlockInputs, BlockProof};
use miden_remote_prover_client::RemoteProverClientError;
use miden_remote_prover_client::remote_prover::block_prover::RemoteBlockProver;
use miden_remote_prover_client::remote_prover::RemoteBlockProver;
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we also need the remote_prover namespace? That is, could this be:

use miden_remote_prover_client::RemoteBlockProver;

Or is this not a good idea for some reason?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated

@sergerad sergerad requested a review from bobbinth February 2, 2026 03:35
@Mirko-von-Leipzig Mirko-von-Leipzig merged commit 925b223 into next Feb 2, 2026
7 checks passed
@Mirko-von-Leipzig Mirko-von-Leipzig deleted the sergerad-mod-nesting branch February 2, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog This PR does not require an entry in the `CHANGELOG.md` file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants