-
Notifications
You must be signed in to change notification settings - Fork 13
Closed
Labels
P3P3 item - no one uses this most probably, but in the end it's better to implement itP3 item - no one uses this most probably, but in the end it's better to implement itgood first issueGood for newcomersGood for newcomers
Milestone
Description
A lot of defined types are pub
for no reason. Many of them could probably be pub(crate)
(or even pub(self)
). Currently, the only items that need to be pub
are ones that are either:
- a part of public C API
- used in the doctests
Maybe if we add Rust integration testing module in the future, there will be more items that need to be pub
, but currently I don't see a reason not to decrease the visibility.
wprzytula
Metadata
Metadata
Assignees
Labels
P3P3 item - no one uses this most probably, but in the end it's better to implement itP3 item - no one uses this most probably, but in the end it's better to implement itgood first issueGood for newcomersGood for newcomers