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.

Sign up for GitHub

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

Jump to bottom

Remove usage of if __name__ == "__main__" from all source files #736

Closed
pietermarsman opened this issue Mar 21, 2022 · 2 comments · Fixed by #974
Closed

Remove usage of if __name__ == "__main__" from all source files #736

pietermarsman opened this issue Mar 21, 2022 · 2 comments · Fixed by #974
Labels
status: accepted type:deprecate type: development Related to the development process for maintainers type: ux Related to the development process of users

Comments

@pietermarsman
Copy link
Member

See #735.

Do this 6 months after is has been deprecated.
@pietermarsman pietermarsman added type: development Related to the development process for maintainers type: ux Related to the development process of users type:deprecate labels Mar 21, 2022
@hholst80
Copy link

So the main code should be moved over to __main__.py files? Good first issue for anyone that would like to get involved!

@pietermarsman
Copy link
Member Author

Currently we use the tools directory for all the commands. But I expect most commands can just be removed since they are for testing purposes only.

Indeed, good first issue.

@pietermarsman pietermarsman mentioned this issue Jul 7, 2024
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
status: accepted type:deprecate type: development Related to the development process for maintainers type: ux Related to the development process of users
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove deprecated files and functionalities pdfminer/pdfminer.six
2 participants
@pietermarsman @hholst80