yt geotiff tidying - Githubissues

ruithnadsteud / yt_georaster

A package for handling geotiff files and georeferenced datasets within yt.

1 stars 2 forks source link

yt geotiff tidying #16

Closed arevill2 closed 3 years ago

arevill2 commented 3 years ago

Check code
Check for PEP 8 compliance
Edit README.md file to detail recent development and provide working examples.

brittonsmith commented 3 years ago

@arevill2, this is great! I'll give it a read through now. Can you push one more change to this PR that uncomments line 156 from .circleci/config.yml that reads # - lint? This will re-enable the flake8 checking during the tests.

arevill2 commented 3 years ago

@arevill2, this is great! I'll give it a read through now. Can you push one more change to this PR that uncomments line 156 from .circleci/config.yml that reads # - lint? This will re-enable the flake8 checking during the tests.

@brittonsmith

arevill2 commented 3 years ago

@arevill2, this is great! I'll give it a read through now. Can you push one more change to this PR that uncomments line 156 from .circleci/config.yml that reads # - lint? This will re-enable the flake8 checking during the tests.

@brittonsmith this sems to cause an issue with the circle/ci checks. I have tried adding the ":" (i.e. - lint:) but does not make any difference. All suggested edits to the README.md have now been applied. Let me know if you would like me to merge once the circle/ci issue is resolved.