From Graham Marlow to ~mgmarlow/public-inbox
Awesome. I merged the patch for version 1.1.0. Thanks again!
From Graham Marlow to ~mgmarlow/public-inbox
I went ahead and made a few edits myself. Would you mind testing them
out to see if it works for you? The new defcustom is named
flymake-clippy-bin-args.
Once verified I'll push up a new version.
---
Introduce a new defcustom for providing arguments to the cargo clippy
binary.
Co-authored-by: Jens Östlund <hi@iensu.me>
Co-authored-by: Graham <info@mgmarlow.com>
---
[message trimmed]
From Graham Marlow to ~mgmarlow/public-inbox
> Hi! I added a variable to be able to pass arguments to the clippy command and I thought it might be useful for others as well :) Hello! Thanks for the contribution. I think this is a great feature. > flymake-clippy.el | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/flymake-clippy.el b/flymake-clippy.el > index e9dfffc..1599896 100644 > --- a/flymake-clippy.el > +++ b/flymake-clippy.el > @@ -65,6 +65,11 @@ > (defvar-local flymake-clippy--proc nil > "Clippy subprocess object, used to ensure obsolete processes aren't reused.")
From Graham Marlow to ~mgmarlow/public-inbox
> Instead of: > > ;;;###autoload > (dolist (file '("deno.json" deno.jsonc")) > (add-to-list 'project-vc-extra-root markers file)) > > You should wrap the form in with-eval-after-load: > > ;;;###autoload > (with-eval-after-load "project" > (dolist (file '("deno.json" deno.jsonc")) > (add-to-list 'project-vc-extra-root markers file))) > > That should do the right thing.
From Graham Marlow to ~mgmarlow/public-inbox
> > Your autoloads file will fail to load with the following error: > > void-function project-vc-extra-root-markers Great catch, thank you. I just pushed your fix to main.
From Graham Marlow to ~mgmarlow/public-inbox
Great, thanks! I've applied your patch: https://git.sr.ht/~mgmarlow/flymake-clippy/commit/d0774403fe96d88bd629d0825ffca46a1786d697. I also bumped the version to 1.0.1 so it should be available the next time MELPA scrapes the repo.
From Graham Marlow to ~mgmarlow/public-inbox
On 8/24/23 4:25 AM, Eugeny Volobuev wrote: > > Looks like in nightly clippy (0.1.73 (180dffba 2023-08-14)) error > messages format were changed. Good catch and thanks for the patch. It looks like that's the error number associated with the lint, though it appears optional in some circumstances: error: expected one of `!` or `::`, found ... --> src/main.rs:20:9 ...
From Graham Marlow to ~mgmarlow/public-inbox
Thank you for the patch, I've applied it to main. I had incorrectly thought vc-use-package was shipping with Emacs 29 so your updates are very helpful. Unfortunate that there's a breaking API change between vc-use-package and the :vc keyword in Emacs main. Best, Graham
From Graham Marlow to ~protesilaos/denote
On 6/25/23 6:40 PM, Jeffrey Fisher wrote: > Thank you to Denote contributors for making such a useful, simple, and > well-documented notes package. > > There is some custom behavior I'd like but can't figure out how to > achieve it. > > I have a subdirectory of my Denote folder called `library/`, and I want > a convenience command for adding notes to that directory. > > I also want to automatically apply the keyword "unread", so that I don't > have to manually type "unread" every time. > > Here is my current code. It creates the note in the correct directory,
From Graham Marlow to ~pkal/public-inbox
> This was the part that confused me. Unless `availableMonospaceFonts' > was some kind of special variable, which to my understanding it is not, > then this just defers the issue to the definition of that variable. Hah, yes sorry. That variable is just a hard-coded list of monospace fonts, filtered by the technique in the aforementioned SO post to only include the ones available on the client's OS. > I'll take a look at your code, and with your permission perhaps adapt it > at some point to work with my configurator? Absolutely, please do. > Another thing I just noticed, as you are assuming Emacs 29, you can use