Use `pub(crate)` instead of `pub` #256

TimJentzsch · 2023-04-09T21:51:11Z

Closes #253.

This PR find-and-replaces pub with pub(crate) (it's a lot).

It also fixes the dead code warnings that emerged from this change.

I recommend to review this by commit to make it a bit easier :)

janhohenheim · 2023-04-10T04:10:33Z

Thanks!
Congrats for all the LOC attributed to you now 😜

Former-commit-id: d1d4769

TimJentzsch added 2 commits April 9, 2023 23:41

Replace pub by pub(crate)

92ba27c

Fix dead code warnings

0024b24

janhohenheim merged commit d1d4769 into janhohenheim:main Apr 10, 2023

janhohenheim added a commit that referenced this pull request Feb 4, 2024

Merge pull request #256 from TimJentzsch/253-pub-crate

7819659

Former-commit-id: d1d4769

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Use `pub(crate)` instead of `pub` #256

Use `pub(crate)` instead of `pub` #256

TimJentzsch commented Apr 9, 2023 •

edited

Loading

janhohenheim commented Apr 10, 2023 •

edited

Loading

Use pub(crate) instead of pub #256

Use pub(crate) instead of pub #256

Conversation

TimJentzsch commented Apr 9, 2023 • edited Loading

janhohenheim commented Apr 10, 2023 • edited Loading

Use `pub(crate)` instead of `pub` #256

Use `pub(crate)` instead of `pub` #256

TimJentzsch commented Apr 9, 2023 •

edited

Loading

janhohenheim commented Apr 10, 2023 •

edited

Loading