Closed nbiederbeck closed 4 years ago
Hi @nbiederbeck!
It truly pains me to reject a well-thought-through PR, but I think the best course of action for this package is to not implement a CLI.
Limiting this project's scope makes contribution management and updates fairly easy. For example, if arXiv makes a breaking change to their API, I can roll a new major release of this package that introduces the same breaking change, without independently having to make design decisions ("do I want to remove this CLI functionality, or do I want to shoehorn the new API behavior to support it?").
Other projects (e.g. https://pypi.org/project/arxiv-cli or https://pypi.org/project/pyarxiv/) implement the download functionality proposed here.
If I were to build out a CLI here, I'd prefer something total (i.e. something with query support) to something partial (just downloads). Similarly, I'd try to expose the various download()
options as CLI arguments.
Thanks for doing this work and opening this PR. I think this may be better-suited for a standalone package depending on this one.
Cheers, Lukas
Description
This is a simple script to download the pdf files from arxiv from the command line, eg:
click
as a new dependency.Note that this is still error prone, eg no checks if query returns results.
If scripts like this are welcome, I'm definitely open to improve this one.
Breaking changes
none
Relevant issues
Checklist
python setupy.py test
.README.md
.