-
Notifications
You must be signed in to change notification settings - Fork 1
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
Format code #53
base: develop
Are you sure you want to change the base?
Format code #53
Conversation
This is a great request. I mainly hesitated to enforce the linting, because of URIs in docstrings, such as:
So all in all: Great initiative, highly appreciated. As discussed in #54 (comment), I have a few more things that I like to add. Therefore I'd suggest to wait with this just for a little and then do the reformatting in a single pass. |
Cheers! :) Black did not touch these long docstrings, but fixes many other things. If you are worried about merge conflicts, just accept your own changes and let pre-commit handle reformatting everything again. That way, these changes can get merged quickly for the CI benefit. |
# Conflicts: # requirements_dev.txt # setup.py # timeseriesx/__init__.py # timeseriesx/base/timestamp_series.py # timeseriesx/mixins/time_zone.py # timeseriesx/mixins/unit.py
# Conflicts: # setup.py # tests/test_timeseriesx.py # timeseriesx/base/timestamp_series.py
Closes #52.