Skip to content
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

Fix emacs native compilation warning #136

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sdwolfz
Copy link

@sdwolfz sdwolfz commented Sep 11, 2022

To reproduce, have emacs built with native compilation and notice the compilation logs. You can then open the offending file and run M-x emacs-lisp-native-compile-and-load before and after the changes to see the warning is removed.

 ■  Warning (comp): lsp-treemacs.el:652:2: Warning: docstring wider than
80 characters
 ■  Warning (comp): lsp-treemacs.el:982:2: Warning: docstring wider than
80 characters
 ■  Warning (comp): lsp-treemacs.el:992:2: Warning: docstring wider than
80 characters

Also fixes some eask specific warnings:

Unmatched website URL 
’[https://github.com/emacs-lsp/lsp-treemacs’](https://github.com/emacs-lsp/lsp-treemacs%E2%80%99); 
add (website-url "https://github.com/emacs-lsp/lsp-treemacs") to Eask-file

Unmatched keyword ’languages’; add (keywords ... "languages") to
Eask-file or consider removing it

There are also other emacs 29 warnings present but those need to be fixed in treemacs.

To reproduce, have emacs built with native compilation and notice the
compilation logs. You can then open the offending file and run `M-x
emacs-lisp-native-compile-and-load` before and after the changes to see
the warning is removed.

```
 ■  Warning (comp): lsp-treemacs.el:652:2: Warning: docstring wider than
80 characters
 ■  Warning (comp): lsp-treemacs.el:982:2: Warning: docstring wider than
80 characters
 ■  Warning (comp): lsp-treemacs.el:992:2: Warning: docstring wider than
80 characters
```
```
Unmatched website URL ’https://github.com/emacs-lsp/lsp-treemacs’; add
(website-url "https://github.com/emacs-lsp/lsp-treemacs") to Eask-file

Unmatched keyword ’languages’; add (keywords ... "languages") to
Eask-file or consider removing it
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant