~mgmarlow

https://mgmarlow.com

~mgmarlow/public-inbox

Last active 7 months ago
View more

Recent activity

Re: [PATCH flymake-clippy] Allow passing arguments to the clippy command 7 months ago

From Graham Marlow to ~mgmarlow/public-inbox

Awesome. I merged the patch for version 1.1.0. Thanks again!

Re: [PATCH flymake-clippy] Allow passing arguments to the clippy command 7 months ago

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]

Re: [PATCH flymake-clippy] Allow passing arguments to the clippy command 7 months ago

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.")

Re: [BUG] autoloads should not assume project.el is loaded 9 months ago

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.

Re: [BUG] autoloads should not assume project.el is loaded 9 months ago

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.

Re: [PATCH] fix regex for nighly clippy 9 months ago

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.

Re: [PATCH] fix regex for nighly clippy 9 months ago

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

...

Re: [PATCH] Update the README with more recent instructions 10 months ago

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

Re: Question: Create a note with automatically applied keyword(s) 11 months ago

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, 

Re: Emacs Config Generator 1 year, 7 days ago

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