diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-08-16 00:59:54 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-08-16 01:03:45 +0900 |
| commit | 3a388b98a9aa6a14b1c7f7dc909c109cf9837800 (patch) | |
| tree | 593e342313c6e6bd3ec2943a1690b8798e0a83ae /crates/shirabe/src/repository/vcs/fossil_driver.rs | |
| parent | aad468e8b75ffc3e87ea6dfa22c53a8299fc08da (diff) | |
| download | php-shirabe-3a388b98a9aa6a14b1c7f7dc909c109cf9837800.tar.gz php-shirabe-3a388b98a9aa6a14b1c7f7dc909c109cf9837800.tar.zst php-shirabe-3a388b98a9aa6a14b1c7f7dc909c109cf9837800.zip | |
refactor: narrow pub(crate) items to private
Porting mapped every PHP `protected` member onto `pub(crate)`, which is
wider than nearly all of them need. Each item demoted here is reached
only from the module that defines it, so the crate-wide visibility
conveyed nothing.
Every `pub(crate)` that survives has at least one reader in another
module of the same crate.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/repository/vcs/fossil_driver.rs')
| -rw-r--r-- | crates/shirabe/src/repository/vcs/fossil_driver.rs | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/crates/shirabe/src/repository/vcs/fossil_driver.rs b/crates/shirabe/src/repository/vcs/fossil_driver.rs index 9a0c4b2e..8ce479fd 100644 --- a/crates/shirabe/src/repository/vcs/fossil_driver.rs +++ b/crates/shirabe/src/repository/vcs/fossil_driver.rs @@ -19,12 +19,12 @@ use shirabe_php_shim::{ #[derive(Debug)] pub struct FossilDriver { - pub(crate) inner: VcsDriverBase, - pub(crate) tags: Option<IndexMap<String, String>>, - pub(crate) branches: Option<IndexMap<String, String>>, - pub(crate) root_identifier: Option<String>, - pub(crate) repo_file: Option<String>, - pub(crate) checkout_dir: String, + inner: VcsDriverBase, + tags: Option<IndexMap<String, String>>, + branches: Option<IndexMap<String, String>>, + root_identifier: Option<String>, + repo_file: Option<String>, + checkout_dir: String, } impl FossilDriver { @@ -95,7 +95,7 @@ impl FossilDriver { Ok(()) } - pub(crate) fn check_fossil(&self) -> anyhow::Result<()> { + fn check_fossil(&self) -> anyhow::Result<()> { let mut ignored_output = String::new(); if self.inner.process.borrow_mut().execute_args( ["fossil", "version"].map(|s| s.to_string()).as_ref(), @@ -112,7 +112,7 @@ impl FossilDriver { Ok(()) } - pub(crate) fn update_local_repo(&mut self) -> anyhow::Result<()> { + fn update_local_repo(&mut self) -> anyhow::Result<()> { assert!(self.repo_file.is_some()); let mut fs = Filesystem::new(None); |
