-
Notifications
You must be signed in to change notification settings - Fork 14
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
Stop using pkg_resources
#50
Conversation
probably could remove the file as well
Pull Request Test Coverage Report for Build 5559304726
💛 - Coveralls |
@@ -61,7 +60,7 @@ | |||
# built documents. | |||
# | |||
# get version using setuptools-scm | |||
release = get_distribution("pydecorate").version | |||
release = get_version("pydecorate") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm tempted to say maybe this should just be import pydecorate; pydecorate.__version__
, but eh let's see how using this goes. I've never used it here.
font_path = resource_filename("pydecorate.fonts", "DejaVuSerif.ttf") | ||
return aggdraw.Font("black", font_path, size=16) | ||
font_path = HERE / "fonts" / "DejaVuSerif.ttf" | ||
return aggdraw.Font("black", font_path.as_posix(), size=16) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aggdraw needs a posix-y path for this? I've never used aggdraw on Windows so I wasn't sure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. I don't feel strongly enough to change it. It does, at a glance, make a reader of the code think that Windows paths are supported. Let's merge this! Thanks.
Resolves #49