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.

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

ingest_tileset probably does not work with ftp #99

Open
mccalluc opened this issue Mar 5, 2019 · 1 comment
Open

ingest_tileset probably does not work with ftp #99

mccalluc opened this issue Mar 5, 2019 · 1 comment

Comments

@mccalluc
Copy link
Collaborator

mccalluc commented Mar 5, 2019

This will never evaluate to True: The character count is wrong.
startswith would be better... or the three if-thens could be replaced with a single regex.

@mildewey
Copy link
Contributor

mildewey commented Mar 6, 2019

Looks like a pretty clear case of copy-paste error. I've fixed it in PR 100, but I don't have an ftp server handy to test against. @mccalluc do you know where we could get one? If we can find a stable one, I'd be happy to amend a test for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants