Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ernestoarbitrio
Copy link
Member

fix: #228

@ernestoarbitrio ernestoarbitrio added the enhancement New feature or request label Feb 15, 2021
@ernestoarbitrio ernestoarbitrio self-assigned this Feb 15, 2021
@coveralls
Copy link

coveralls commented Feb 15, 2021

Coverage Status

Coverage remained the same at 100.0% when pulling 790aa07 on enable-logging-in-tiler into 4f59b9a on master.

Copy link
Collaborator

@alessiamarcolini alessiamarcolini left a comment

Choose a reason for hiding this comment

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

please update docstrings according to your changes

Copy link
Collaborator

@alessiamarcolini alessiamarcolini left a comment

Choose a reason for hiding this comment

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

suggested change only for one, please update the others as well

Copy link
Collaborator

@alessiamarcolini alessiamarcolini left a comment

Choose a reason for hiding this comment

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

LGTM

@alessiamarcolini alessiamarcolini merged commit 7036066 into master Feb 16, 2021
@alessiamarcolini alessiamarcolini deleted the enable-logging-in-tiler branch February 16, 2021 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add "verbose" level argument in "GridTiler.extract" method

4 participants