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

Update Vale to latest #155

Closed
wants to merge 6 commits into from
Closed

Update Vale to latest #155

wants to merge 6 commits into from

Conversation

dennisameling
Copy link
Member

@dennisameling dennisameling commented Sep 14, 2023

We're quite a few versions behind. Let's go from 2.20.2 (Sept 14 2022) to 2.28.3. Changelog: https://github.com/errata-ai/vale/releases

This also fixes the Gitpod/VS Code integration by bumping the Esbonio language server to the latest version 🚀

image

@@ -5,5 +5,6 @@
"esbonio.sphinx.confDir": "${workspaceFolder}/docs",
"restructuredtext.pythonRecommendation.disabled": true,
"restructuredtext.preview.name": "sphinx",
"gitlens.showWhatsNewAfterUpgrades": false
"gitlens.showWhatsNewAfterUpgrades": false,
"restructuredtext.linter.rstcheck.executablePath": "rstcheck"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes the rstcheck vscode integration. Source: vscode-restructuredtext/vscode-restructuredtext#400 (comment)

However, the maintainers mention that rstcheck shouldn't be necessary anymore as Esbonio reports most of the issues too: vscode-restructuredtext/vscode-restructuredtext#400 (comment). I'm happy to remove it altogether but Esbonio certainly doesn't catch everything yet.

@RCheesley
Copy link
Member

When I tried to load this on Gitpod, I got the following in a popup:

Some workspace/folder-level settings ("restructuredtext.linter.rstcheck.executablePath") from the untrusted workspace are disabled by default. If this workspace is trusted, explicitly enable the workspace/folder-level settings by running the "reStructuredText: Toggle Workspace Trust Flag" command.

I have to select okay to proceed.

Hopefully this also means we can close this PR which I was not able to get to actually work properly, as it kept throwing errors.

If the first comment is okay then this is good to go!

@RCheesley
Copy link
Member

Going to close this one as we've done it in other PRs which didn't break stuff 🚀

@RCheesley RCheesley closed this Feb 2, 2024
@RCheesley RCheesley deleted the update-vale-to-latest branch February 2, 2024 09:31
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.

None yet

2 participants