Skip to content

Vale prose linter issues #1298

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

Closed
omfgnuts opened this issue Jan 27, 2020 · 18 comments · Fixed by #1338
Closed

Vale prose linter issues #1298

omfgnuts opened this issue Jan 27, 2020 · 18 comments · Fixed by #1338

Comments

@omfgnuts
Copy link

omfgnuts commented Jan 27, 2020

Description

So, I happily moved from VS Code to code-server and my primary extension (Vale) doesn't work, I suspect it's due to some sandboxing issues, but I'm not sure.

Steps to Reproduce

  1. Download Vale v 1.7.1 (https://github.com/errata-ai/vale/releases)
  2. Download code-server 1.41.1
  3. Drop Microsoft latest StyleGuide (https://github.com/errata-ai/Microsoft/releases) somewhere. i.e /home/%username%/Work/ci/styleguide
  4. Create .vale.ini i.e /home/%username%/Work
  5. Enter following info there
# Don't point this config directly to Microsoft folder, point it one directory before
#, i.e your path to styleguide is ci/styles/Microsoft, point this config to ci/styles

StylesPath = path/to/styles/folder
MinAlertLevel = suggestion # suggestion, warning or error

# Only Markdown and .txt files; change to whatever you're using.
[*]

# List of styles to load.
BasedOnStyles = Microsoft
  1. Create .md document and write there "- Knowledge of OOP;" (OOP should trigger a linting error)
  2. Nothing works 🤷‍♂
@nhooyr nhooyr added the waiting-for-info Waiting for more information from submitter label Jan 27, 2020
@nhooyr
Copy link
Contributor

nhooyr commented Jan 27, 2020

Nothing works 🤷‍♂

Could you please elaborate on what you mean? Does linting throw an error?

@omfgnuts
Copy link
Author

omfgnuts commented Jan 29, 2020

Nope, no errors, it just doesn't lint anything (0 errors in both vscode and chrome dev tools)

I instantly tried regular vs code and it worked like a charm.

markdownlint, plantuml, and other extensions work just perfect

@sr229
Copy link
Contributor

sr229 commented Jan 29, 2020

Try relaunching the binary with verbose flag and send us the logs, thank you!

@nhooyr
Copy link
Contributor

nhooyr commented Jan 29, 2020

@omfgnuts

I'll investigate this and let you know if I can get it to work.

@nhooyr nhooyr added needs-investigation and removed waiting-for-info Waiting for more information from submitter labels Jan 29, 2020
@omfgnuts
Copy link
Author

@nhooyr Thank you! I really appreciate your efforts.

@sr229 Um, some magic just happened, I did literally (really, I did nothing) nothing besides launching code-server with --verbose, and all of a sudden Vale worked! I managed to take a screenshot as a proof for Vale Devs & you guys. After taking a screenshot I happily relaunched code-server just to find out that it stopped working again, bummer. So the issue is definitely still somewhere there.

As for logs, both --verbose or --log not showing anything (is there a specific place code-server saves the logs?)

@ChrisChinchilla
Copy link

Hey all, I maintain the VSCode vale extension and am also looking into this now, but I am not familiar with code-server, so will take some experimentation.

@ChrisChinchilla
Copy link

I'm not sure how you feed extensions into the custom extension marketplace, or which one @omfgnuts installed, but on n first inspection I see that when I search for Vale in the Code Server marketplace, the results return a completely different and very old extension, which even shows a depreciation warning.

Screen Shot 2020-02-02 at 17 43 09

@nhooyr
Copy link
Contributor

nhooyr commented Feb 2, 2020

Thank you for the response @ChrisChinchilla

I'm gonna close this for #1299 which is the tracking issue to improve the extension scraping process.

@nhooyr nhooyr closed this as completed Feb 2, 2020
@omfgnuts
Copy link
Author

omfgnuts commented Feb 3, 2020

I've installed the latest .vsix manually, not from the built-in marketplace (as it's been specifically noted in my initial post), why close issue report? It's not yet clear how to make it work permanently, there's a solution for sure, as it worked a couple of times, we just need to find what made it functional. @nhooyr

@ChrisChinchilla
Copy link

Sorry didn't see anything in original report about installing VSIX file… I think this is still highly likely because of browser sandboxing but will investigate again when I have time

@sr229 sr229 reopened this Feb 3, 2020
@sr229
Copy link
Contributor

sr229 commented Feb 3, 2020

Reopened because of request.

@jdkato
Copy link

jdkato commented Feb 6, 2020

I followed the repro instructions on macOS (both via Docker and the binary) and I was able to get it working permanently (from what I can tell):

(Vale v1.7.1 running on code-server via Docker.)

  • When using the macOS binary, it worked out-of-the-box after I manually installed the extension.

  • When using Docker, I had to copy a Linux build of Vale into my workspace (${workspaceFolder}/bin) and then update the vscode-vale.path setting to ${workspaceFolder}/bin/vale.

I think the best first debugging step here is to determine the output of which vale in code-server's terminal. If it's empty, you'll likely need to manually set the path as I had to do on Docker.

@omfgnuts
Copy link
Author

omfgnuts commented Feb 6, 2020

omfgnuts@penguin:~/Work$ which vale
/usr/bin/vale

It's weird that it randomly works, but can't quite get WHY :(

@omfgnuts
Copy link
Author

Does anyone have any ideas about how to debug this issue further?

@omfgnuts
Copy link
Author

I'm glad that it worked for macOS, but I'm not on macOS; is there any debug option I can turn on, maybe I should strace-launch code-server?

@code-asher
Copy link
Member

We have version 0.9.1 of the extension in our marketplace now. I tried it out and it shows warnings/errors in the problems panel but not in the file itself.

It's fixed in a PR: #1338 so hopefully we can get it merged soon and get out a new release.

@code-asher code-asher mentioned this issue Feb 25, 2020
6 tasks
@omfgnuts
Copy link
Author

omfgnuts commented Mar 6, 2020

@code-asher I can't even express how grateful I am, thank you so much

@ChrisChinchilla
Copy link

@code-asher Great thanks! I'll try to remember to test future updates with code server too…

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 a pull request may close this issue.

6 participants