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
It would be interesting to run this on the Lurk code base and see if there are pub annotations that do not need to be there.
Caution
We should try to avoid breaking public dependents needlessly. When in doubt, use the #[deprecated(since = "...")] attribute rather than remove blindly. Note this script will turn up false positives, a typical example being error types which need to be public to surface errors from your API.
Warning
rename 'gfind' to 'find' and 'gsed' to 'sed' in the script before using if you're on a Linux machine where gnu variants of these utilities are the default.
huitseeker
changed the title
Try to remove uneeded pub modifiers through brute-force
[LUR-25] Try to remove uneeded pub modifiers through brute-force
Jan 5, 2024
In the course of making really sure the visibility modifiers in Arecibo were set as tightly as possible, I've written a small script to brute-force that:
https://gist.github.com/huitseeker/99ba921da591497de7879d1724e3e96b
See some selected results in lurk-lang/arecibo#239
It would be interesting to run this on the Lurk code base and see if there are
pub
annotations that do not need to be there.Caution
We should try to avoid breaking public dependents needlessly. When in doubt, use the
#[deprecated(since = "...")]
attribute rather than remove blindly. Note this script will turn up false positives, a typical example being error types which need to be public to surface errors from your API.Warning
rename 'gfind' to 'find' and 'gsed' to 'sed' in the script before using if you're on a Linux machine where gnu variants of these utilities are the default.
From SyncLinear.com | LUR-25
The text was updated successfully, but these errors were encountered: