You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
we can define the default visibility of delta internals by prefixing the struct/fn/... with pub(crate) instead of #[cfg_attr(not(feature = "developer-visibility"), visibility::make(pub(crate)))]. This simplifies the codebase.
sebastiantia
changed the title
Replace #[cfg_attr(not(feature = "developer-visibility"), visibility::make(pub(crate)))] with pub(crate)s
simplify internal visibility by replacing conditional flags with pub(crate) prefixes
Jan 31, 2025
we can define the default visibility of delta internals by prefixing the struct/fn/... with
pub(crate)
instead of#[cfg_attr(not(feature = "developer-visibility"), visibility::make(pub(crate)))]
. This simplifies the codebase.Examples:
delta-kernel-rs/kernel/src/snapshot.rs
Line 145 in 06d8dbb
delta-kernel-rs/kernel/src/actions/visitors.rs
Line 21 in 06d8dbb
and many more...
The text was updated successfully, but these errors were encountered: