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

Port Project to Python3.9 #16

Merged
merged 1 commit into from
Feb 7, 2024
Merged

Conversation

jharshman
Copy link
Contributor

Port project to Python3.9

This PR ports this project to Python 3.9.

@jharshman jharshman changed the title Port Project to Python3.9 WIP: Port Project to Python3.9 Jan 9, 2024
Copy link

@mathesond2 mathesond2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

assetman/manifest.py Outdated Show resolved Hide resolved
from assetman.parsers.django_parser import DjangoParser
return DjangoParser(template_path, settings)
print(template_type, template_path)
from assetman.parsers.tornado_parser import TornadoParser
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we move this import to the top now since we don't need to dynamically import?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like the dynamic import might be there because there is a circular dependency between assetman/tools.py and assetman/parsers/tornado_parser.py

@jharshman jharshman changed the title WIP: Port Project to Python3.9 Port Project to Python3.9 Jan 17, 2024
Resolves: TS-2847
@jehiah jehiah merged commit d3820bf into bitly:master Feb 7, 2024
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants