-
-
Notifications
You must be signed in to change notification settings - Fork 645
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
Sideloader: handle binary files, support multiple directories #3041
Conversation
80561b9
to
54340cb
Compare
Oops there's actually another important change in there, I should split that up. (serving from multiple directories) |
024e3f6
to
7b44d85
Compare
cider-eval.el
Outdated
(defvar cider-sideloader-dir (file-name-directory load-file-name)) | ||
(defvar cider-sideloader-dirs | ||
(list (file-name-directory load-file-name)) | ||
"Directories where we look for resources requested by the sideloader") |
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 think the missing .
is breaking the CI.
Wow! 🤣
Seems you've address my comment from the meta issue already. :D Probably we should make this eventually a defcustom to which we always append CIDER's source dir + lib (or something along those lines). Seems the win build has hit some CircleCI limit... No love for OSS. :-) |
Make sure that files are served byte-for-byte as they are on disk, without emacs doing any coding system conversion. This also prevents base64-encode from complaining about multibyte characters. Support multiple directories as sources for the sideloader, e.g. cider-nrepl/src, cider-nrepl/resources, orchard/src
443a822
to
be26e2d
Compare
The linting is fixed. The windows builds are still red but that just seems to be a CircleCI limit we've hit. |
That's pretty weird, as supposedly we should get a pretty high limit on an OSS plan https://circleci.com/open-source/ I guess I'll have to contact their support (if they provide any for the OSS projects). |
My contact at CircleCI says the limits for Windows builds are different since they're more expensive. Seems we should use them sparingly.
|
Make sure that files are served byte-for-byte as they are on disk, without emacs
doing any coding system conversion. This also prevents base64-encode from
complaining about multibyte characters.
Support multiple directories as sources for the sideloader, e.g.
cider-nrepl/src, cider-nrepl/resources, orchard/src
Part of #3037
Discovered this because cider.nrepl.middleware.track-state contains a U+2019 RIGHT SINGLE QUOTATION MARK :)
eldev test
)eldev lint
) which is based onelisp-lint
and includescheckdoc
, check-declare, packaging metadata, indentation, and trailing whitespace checks.