deprecate mapbox-upload in favor of mapbox-cli-py?
mapsam opened this issue · 3 comments
mapsam commented
It'd be good to deprecate one of our CLI upload tools, mapbox-upload, so we can focus development and fixes in a single repo. Mapbox-cli-py is our go-to CLI tool, and should be able to do all of the work mapbox-upload does.
On first glance, it doesn't do anything more than this CLI, so it shouldn't be a huge lift. I'll do more digging to make sure all features are carried over.
willwhite commented
Yes, sounds like a great idea to me. Thanks for taking this on.
…On Wed, Dec 21, 2016 at 19:33 Sam Matthews ***@***.***> wrote:
It'd be good to deprecate one of our CLI upload tools, mapbox-upload
<https://github.com/mapbox/mapbox-upload>, so we can focus development
and fixes in a single repo. Mapbox-cli-py is our go-to CLI tool, and should
be able to do all of the work mapbox-upload does.
On first glance, it doesn't do anything more than this CLI, so it
shouldn't be a huge lift. I'll do more digging to make sure all features
are carried over.
cc @rclark <https://github.com/rclark> @willwhite
<https://github.com/willwhite> @perrygeo <https://github.com/perrygeo>
@mattficke <https://github.com/mattficke> @sgillies
<https://github.com/sgillies>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#83>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAFtfPvr0vxyDfr0jihsvtWo9tfMjp9nks5rKcVOgaJpZM4LThbm>
.
sgillies commented
@mapsam shall we enumerate the features mapbox-cli-py upload lacks?
- upload from S3 as well as local file
- progress bar (possible with http://click.pocoo.org/5/utils/#showing-progress-bars)
- ?
sgillies commented
I've checked the usage of mapbox-upload
around various projects and after we ship version 0.6 of mapbox-cli-py migration, migration of a bash script won't be much more than s/mapbox-upload/mapbox upload/g
.