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
Download script #22
Merged
Download script #22
Changes from 2 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
28c17a2
WIP download script
newsch 7254bc3
Add requested changes
newsch 3a4d121
Canonicalize input paths
newsch 9ee1e8d
Fix check for uninitialized variable
newsch bae03b9
Improve comments
newsch fe295b2
Fix jq list output
newsch 0a1e059
Use real enterprise dump url
newsch 27ff9cb
Track number of missing dumps
newsch 54727b9
Working downloads
newsch bce44d1
Store in subdirs
newsch 5077ed0
Document usage
newsch 4c2c6e9
Rename temp dir
newsch af80f2a
Check for DUMP_DIR existence
newsch 98d5a8a
Mention download.sh in README
newsch 30b19ca
Fix typo
newsch 187e294
Fix typo typo
newsch b06167f
Clarify LANGUAGES parsing
newsch 38faebb
Remove old workaround for lack of pipefail
newsch 82f2993
Remove extra whitespace
newsch 9d2d2e5
Add option to delete old dumps
newsch 4d91992
Use wget2 by default
newsch 5c8be74
Write requested help to stdout
newsch e7b5c19
Fix link replacement
newsch cf1ac05
Make base url configurable
newsch 4be00cd
Make concurrent downloads configurable
newsch f08fd7d
Clarify -c behavior
newsch b118724
Use custom user agent with email
newsch b21a999
Fix typos
newsch File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can spaces be added here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't seen an example with spaces in the name. All of the browser user agents use CamelCase instead of spaces.