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
Show notifications about fetch library issues #7868
Conversation
User-facing changes: The editor will now show notifications about failed attempts to fetch libraries. Fixes #4000
(defn copy-engine-executable! ^File [^File target-dmengine platform {:keys [^File dmengine ^File engine-archive] :as engine-descriptor}] | ||
(assert (or dmengine engine-archive)) | ||
(cond | ||
(some? dmengine) | ||
(fs/copy-file! dmengine target-dmengine) | ||
|
||
(some? engine-archive) | ||
(unpack-dmengine! engine-archive (dmengine-filename platform) target-dmengine)) | ||
target-dmengine) | ||
|
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.
Unused.
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.
Looks good to me!
@@ -427,7 +427,8 @@ | |||
Options: | |||
:fail :silently will not throw an exception on failure, and instead return an empty vector. | |||
:target :keep will not replace the target file if it exists. | |||
:target :replace (default) will replace the target file if it exists." | |||
:target :replace (default) will replace the target file if it exists. | |||
Returns a vector of source+target pairs if successful" |
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.
source+target File pairs
?
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 :)
User-facing changes:
The editor will now show notifications about failed attempts to fetch libraries.
Fixes #4000