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

Start with Markdownlint for vscode editing #1377

Merged
merged 1 commit into from
Dec 25, 2020

Conversation

Confectrician
Copy link
Contributor

Including some initial linting error fixes as showcase.

This is on the todo list for a very long time.
We have discussed about several solutions with several possible editors.

Since vscode has become quite popular meanwhile and many people are using it anyway for their openHAB environments i want to give it another try.
VSCode can handle Markdown editing by default in a basic matter.

This PR introduces a custom configuration for the https://marketplace.visualstudio.com/items?itemName=DavidAnson.vscode-markdownlint extension.
It will apply some default linting with custom rules we have configured in this repo on top.

This way we can make sure that everyone uses the same ruleset and we can establish some kind of markdown standard for the future.

The handling was pretty easy so far.
There is a set of standard fixes that can be applied automatically with one command.
Mostly there are ony a few markdown errors left, that need some attention.

References: #50 #196 #999

Signed-off-by: Jerome Luckenbach [email protected]

@Confectrician Confectrician changed the title Start with Markdownlint for vscode editing. Start with Markdownlint for vscode editing Dec 25, 2020
@netlify
Copy link

netlify bot commented Dec 25, 2020

✔️ Thanks for your pull request to the openHAB documentation! The result can be previewed at the URL below (this comment and the preview will be updated if you add more commits).

🔨 Explore the source changes: d4a3af2

🔍 Inspect the deploy logs: https://app.netlify.com/sites/openhab-docs-preview/deploys/5fe6199b32319c4512ea5e5c

😎 Browse the preview: https://deploy-preview-1377--openhab-docs-preview.netlify.app

@Confectrician Confectrician marked this pull request as ready for review December 25, 2020 22:44
@Confectrician Confectrician merged commit 953e074 into openhab:main Dec 25, 2020
@Confectrician Confectrician deleted the add-markdownlint branch December 25, 2020 22:44
@Confectrician Confectrician added this to the 3.1 milestone Nov 18, 2022
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