-
-
Notifications
You must be signed in to change notification settings - Fork 769
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make all _
fields private
#333
Comments
I wonder if this would break any workarounds people have implemented in production. In the past I've accessed these via the console to modify things to test and see if it addresses an issue I ran into. |
This is what 2.0 is for... |
Unless the project is using a minifier that mangles private class member names (not common practice), marking members as private shouldn't affect such debug console hacks. |
This would break tons of workaround that people use.... I even have some of these kicking around. |
This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 30 days. |
Oh boy, stale bot. |
Unfortunately, due to the large echo system around maplibre I think this won't help anyone but only cause pain. |
Motivation
The motivation is to properly define all methods and members of classes with their appropriate modifier (public, private mainly).
This is a big challenge that requires a lot of work.
Having said that, doing it for
_
prefixed fields and methods should be relatively simple and an easy win.The text was updated successfully, but these errors were encountered: