Skip to content
This repository was archived by the owner on Feb 5, 2026. It is now read-only.

Read args only when needed to allow importing the module#50

Open
depau wants to merge 4 commits into
forksociety:masterfrom
depau:args
Open

Read args only when needed to allow importing the module#50
depau wants to merge 4 commits into
forksociety:masterfrom
depau:args

Conversation

@depau

@depau depau commented Oct 11, 2017

Copy link
Copy Markdown
Collaborator

The arguments should not be read on top-level code, only when the program is executed.

This way other Python code can use the module functions.

@PrabhanshuAttri

Copy link
Copy Markdown
Collaborator

Thanks @Depaulicious. In future, please make sure that you create an issue before creating a PR. Have you test it for all the functionalities?

@depau

depau commented Dec 21, 2017

Copy link
Copy Markdown
Collaborator Author

In future, please make sure that you create an issue before creating a PR.

Sounds like a weird way of working to me but no problem.

Have you test it for all the functionalities?

Yes

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants