pprevos / emacs-writing-studio

Emacs configuration for authors who research, write and publish articles, books and websites.
https://lucidmanager.org/tags/emacs/
GNU General Public License v3.0
473 stars 30 forks source link

Hiding warnings that may appear during initial setup #3

Closed grtcdr closed 1 year ago

grtcdr commented 1 year ago

Hi there,

I just took this for a spin and noticed that, during setup i.e. after packages are initially installed from their the archives, the *Warnings* buffer proceeded to harass me with trivial non-errors. I think novice Emacs users and authors with no programming experience might find this behavior a bit terrifying.

2023-09-27T12:43:14,894895115+01:00

Would you be open to setting warning-minimum-level to :error? I could open a pull request with the appropriate change if you're willing to move forward with it.

Thank you so much for this package and for all the lovely writer-centric articles you've been publishing.

pprevos commented 1 year ago

Hi Taha,

Great suggestion, I did not know that such a setting existed. Is this something to be set in each package declaration or is there a variable?

Please submit a pull request so you get the credit in the repo.

grtcdr commented 1 year ago

This only needs to be set once, any package or piece of Elisp that attempts to warn the user about something automatically abides by the value warning-minimum-level is set to.

I'll create a pull request shortly, would you prefer the variable be set in its corresponding use-package form or to be placed more generally?

Side note: My name is not Andreas :laughing:

pprevos commented 1 year ago

Apologies Taha, I am mixing-up messages.

Using the use-package macro for one line of code is perhaps a bit much.

Perhaps it best fits with the ews-packages.el module. I include these modules in my web pages so that I keep referential integrity between the code and the published pages. I generate init.el with a script.

grtcdr commented 1 year ago

Apologies Taha, I am mixing-up messages.

No worries :)

Using the use-package macro for one line of code is perhaps a bit much.

Agreed, although I'm guilty of doing this in my own init.el.