-
-
Notifications
You must be signed in to change notification settings - Fork 64
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
Improve the netlify preview to show packages #308
Conversation
Thanks for your pull request, @wilzbach! |
. "$(curl -fsSL --retry 5 --retry-max-time 120 --connect-timeout 5 --speed-time 30 --speed-limit 1024 https://dlang.org/install.sh | bash -s install "dmd-${DMD_VERSION}" --activate)" | ||
. "$(curl "${CURL_FLAGS[@]}" https://dlang.org/install.sh | bash -s install "dmd-${DMD_VERSION}" --activate)" | ||
|
||
curl "${CURL_FLAGS[@]}" https://code.dlang.org/api/packages/dump | gunzip > mirror.json |
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.
We could also upload a JSON dump somewhere else if this becomes a problem, but for now there aren't too many PRs at dub-registry so having a separate file for this will probably just be more work.
source/dubregistry/mirror.d
Outdated
packs = requestHTTP(url ~ InetPath("api/packages/dump")).readJson().deserializeJson!(DbPackage[]); | ||
} | ||
|
||
logInfo("Updated for %s downloaded.", url); |
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.
shouldn't this say updates?
source/dubregistry/mirror.d
Outdated
@@ -71,6 +92,8 @@ nothrow { | |||
logDiagnostic("Full error: %s", e.toString().sanitize); | |||
} | |||
} | |||
|
|||
logInfo("Updated for '%s' successfully processed.", fileOrUrl); |
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.
ditto
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.
Yes. Fixed. Thanks!
73d0c37
to
2c6cf39
Compare
2c6cf39
to
efcc586
Compare
FYI @ others: I'm going to merge this now, s.t. we can play with the preview on the other PRs. As there's no auto-deploy or similar setup, this shouldn't be an issue. |
As mentioned in #307, downloading the Json doesn't work (or well it takes 10 minutes), so I made
--mirror
support importing a manually downloaded copy of the Json file.This is also handy for my local development, when I want to quickly nuke the entire database and on my machine
--mirror=...
doesn't work well either (could be due to my crappy internet though).