-
Notifications
You must be signed in to change notification settings - Fork 5
Cookiecutter update, drop black and isort #525
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
Conversation
…ter conventions, drop old owslib handling code
|
Note It appears that this Pull Request modifies the On inspection, the No further action is required. |
|
Not sure I can review this. I don't know this repo well enough. I can rubber stamp if you are blocked and need this one in, if that is your intention. |
|
Not needed. It would be good to get an actual review on this. There's still one strange error happening, but I hope it isn't something broken upstream. |
Co-authored-by: David Huard <huard.david@ouranos.ca>
Pull Request Checklist:
number) and pull request (:pull:number) has been added.What kind of change does this PR introduce?
cookiecuttertemplate to use newer dependencies and conventionsblackandisort(redundant viaruff)owslibDoes this PR introduce a breaking change?
Somewhat. Code style conventions are different, but code is largely the same, barring a handful of small suggested changes.
Other information:
This probably doesn't require a minor release.