Allow parser.load_fp without module_name #292

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

Open

aisk wants to merge 2 commits into Thriftpy:master from aisk:load_fp_without_module_name

Member

aisk commented Jul 14, 2024

Just like parser.load, allow omitting the module_name to prevent polluting the global sys.modules.


          Allow parser.load_fp without module_name

b2fd698

aisk marked this pull request as draft

July 14, 2024 17:48


          Fix test

b0ff318

aisk marked this pull request as ready for review

July 15, 2024 14:43

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

Labels

None yet