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.

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

Apply Black #537

Merged
merged 17 commits into from
Jun 18, 2021
Merged

Apply Black #537

merged 17 commits into from
Jun 18, 2021

Conversation

wenzeslaus
Copy link
Member

  • Black for vector
  • Black for raster
  • Black for temporal
  • Black for imagery
  • Black for gui
  • Black for general
  • Black for raster3d
  • Black for display
  • Black for db and misc
  • Black for hadoop
  • Add CI for Black
  • Black for tools
  • Config for Black

@wenzeslaus wenzeslaus changed the title apply black Apply Black Jun 2, 2021
@wenzeslaus wenzeslaus mentioned this pull request Jun 2, 2021
Closed
17 tasks
@wenzeslaus wenzeslaus added enhancement New feature or request CI Continuous integration labels Jun 2, 2021
@wenzeslaus
Copy link
Member Author

I will be more comfortable merging this with tests up and running again, so I will wait a bit with merge.

@neteler
Copy link
Member

neteler commented Jun 8, 2021

I will be more comfortable merging this with tests up and running again, so I will wait a bit with merge.

I believe to have fixed it in #538 but not sure how to update this PR to make it realize the fix...

@nilason
Copy link
Contributor

nilason commented Jun 8, 2021

I will be more comfortable merging this with tests up and running again, so I will wait a bit with merge.

I believe to have fixed it in #538 but not sure how to update this PR to make it realize the fix...

Maybe a rebase is needed, to include the changes of #538?

@wenzeslaus wenzeslaus added the needs rebase Rebase to or merge with the latest base branch is needed label Jun 8, 2021
@wenzeslaus wenzeslaus merged commit 6bcda5a into OSGeo:master Jun 18, 2021
@wenzeslaus wenzeslaus deleted the apply-black branch June 18, 2021 12:52
@LorenzoBottaccioli
Copy link

LorenzoBottaccioli commented Jun 24, 2021 •
edited
Loading

In usage, I get this warning

UserWarning: 'is_latlong()' is deprecated. Please use 'crs.is_geographic'.
  warnings.warn("'is_latlong()' is deprecated. Please use 'crs.is_geographic'.")

Moreoverpyproj>2.2has no is_latlong().

@neteler neteler mentioned this pull request Jun 24, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CI Continuous integration enhancement New feature or request needs rebase Rebase to or merge with the latest base branch is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@wenzeslaus @neteler @nilason @LorenzoBottaccioli