Skip to content
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

Remove of prefix _ for python file #12

Open
jmkerloch opened this issue Jan 10, 2023 · 1 comment
Open

Remove of prefix _ for python file #12

jmkerloch opened this issue Jan 10, 2023 · 1 comment
Labels
question Further information is requested

Comments

@jmkerloch
Copy link
Contributor

@azarz @lgrd All python files are prefixed with _.

Is there any reason for that ?

I'm afraid this kind of naming could be misinterpreted by some parser. It may be better to remove all prefix.

@jmkerloch jmkerloch added the question Further information is requested label Jan 10, 2023
@azarz
Copy link
Member

azarz commented Jan 10, 2023

It was because those files are "internal" / "private", and the methods in those modules shout not be imported when one writes

from r2gg import *

However if this causes errors in parsers, we can remove this prefix.

@jmkerloch jmkerloch moved this to 🆕 New in Road2 backlog Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
Status: 🆕 New
Development

No branches or pull requests

2 participants