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

warnings on install #22

Open
7yl4r opened this issue Dec 1, 2023 · 1 comment
Open

warnings on install #22

7yl4r opened this issue Dec 1, 2023 · 1 comment

Comments

@7yl4r
Copy link

7yl4r commented Dec 1, 2023

I am pretty new to emacs. I just walked through install of melpa and then installed as instructed in the README here. After install I get a lot of warnings. These warnings also show when opening a new .qmd.

⛔ Warning (comp): quarto-mode.el:56:22: Warning: ‘make-variable-buffer-local’ not called at toplevel
⛔ Warning (comp): quarto-mode.el:62:20: Warning: ‘make-variable-buffer-local’ not called at toplevel
⛔ Warning (comp): quarto-mode.el:82:2: Warning: custom-declare-variable `quarto-poly-markdown-exporter' docstring wider than 80 characters
⛔ Warning (comp): quarto-mode.el:109:2: Warning: custom-declare-variable `quarto-force-preview' docstring wider than 80 characters
⛔ Warning (comp): quarto-mode.el:154:2: Warning: docstring wider than 80 characters
⛔ Warning (comp): quarto-mode.el:171:2: Warning: docstring wider than 80 characters
⛔ Warning (comp): quarto-mode.el:214:11: Warning: Unused lexical variable `browser-path'
⛔ Warning (comp): quarto-mode.el:236:9: Warning: reference to free variable ‘poly-quarto-mode-map’
⛔ Warning (comp): quarto-mode.el:245:11: Warning: Unused lexical variable `data'
⛔ Warning (comp): quarto-mode.el:245:25: Warning: Unused lexical variable `symbol-status'
⛔ Warning (comp): quarto-mode.el:245:39: Warning: Unused lexical variable `error-thrown'
⛔ Warning (comp): quarto-mode.el:252:2: Warning: docstring wider than 80 characters
⛔ Warning (comp): quarto-mode.el:255:13: Warning: Unused lexical variable `this-project-directory'
⛔ Warning (comp): quarto-mode.el:281:2: Warning: docstring wider than 80 characters

I can figure out how to hide these if they are not a major concern but also it looks like #17 is a draft fix for this, correct?
How can I help push that forward?

@cscheid
Copy link
Contributor

cscheid commented Dec 1, 2023

How can I help push that forward?

You can fork the repo, make sure that the tests pass for #17, and make a PR. Unfortunately, I don't have time to handle this right now.

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

No branches or pull requests

2 participants