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

docs: Lint all *.md files and replace product-edc with tractusx-edc #134

Conversation

florianrusch-zf
Copy link
Contributor

@florianrusch-zf florianrusch-zf commented Mar 21, 2023

To provide some documentation for the https://github.com/eclipse-tractusx/eclipse-tractusx.github.io repository we need to lint our markdown files.

This was done by using the npm package: https://www.npmjs.com/package/markdownlint-cli2

$ npm install -g markdownlint-cli2
$ markdownlint-cli2-config .markdownlint.yaml "**/*.md"

The configuration file .markdownlint.yaml was configured like the lintting is configured in the eclipse-tractusx.github.io repository. The configuration file itself will be added to the repository with the PR #132

"default": true
# Do not restrict line length: https://github.com/DavidAnson/markdownlint/blob/main/doc/Rules.md#MD013
"MD013": false
# Allow same content on headlines on siblings: https://github.com/DavidAnson/markdownlint/blob/main/doc/Rules.md#MD024
"MD024":
  "siblings_only": true

closes #111

@florianrusch-zf
Copy link
Contributor Author

florianrusch-zf commented Mar 21, 2023

Right now there are still some errors in the files. @stephanbcbauer how should we handle the following linting errors?

docs/samples/Transfer Data.md:9:65 MD051/link-fragments Link fragments should be valid [Context: "[chapter 1](#1--optional--local-setup)"]
docs/samples/Transfer Data.md:13:4 MD051/link-fragments Link fragments should be valid [Context: "[(optional) Local Setup](#1--optional--local-setup)"]
README.md:1:1 MD033/no-inline-html Inline HTML [Element: a]
README.md:1 MD041/first-line-heading/first-line-h1 First line in a file should be a top-level heading [Context: "<a name="readme-top"></a>"]
README.md:10:1 MD033/no-inline-html Inline HTML [Element: br]
README.md:11:1 MD033/no-inline-html Inline HTML [Element: div]
README.md:12:3 MD033/no-inline-html Inline HTML [Element: a]
README.md:13:5 MD033/no-inline-html Inline HTML [Element: img]
README.md:16:3 MD033/no-inline-html Inline HTML [Element: h3]
README.md:17:3 MD033/no-inline-html Inline HTML [Element: h4]
README.md:19:3 MD033/no-inline-html Inline HTML [Element: p]
README.md:21:5 MD033/no-inline-html Inline HTML [Element: br]
README.md:22:5 MD033/no-inline-html Inline HTML [Element: a]
README.md:22:79 MD033/no-inline-html Inline HTML [Element: strong]
README.md:23:5 MD033/no-inline-html Inline HTML [Element: br]
README.md:24:5 MD033/no-inline-html Inline HTML [Element: br]
README.md:25:5 MD033/no-inline-html Inline HTML [Element: a]
README.md:27:5 MD033/no-inline-html Inline HTML [Element: a]
README.md:29:5 MD033/no-inline-html Inline HTML [Element: a]
README.md:34:1 MD033/no-inline-html Inline HTML [Element: details]
README.md:35:3 MD033/no-inline-html Inline HTML [Element: summary]
README.md:36:3 MD033/no-inline-html Inline HTML [Element: ol]
README.md:37:5 MD033/no-inline-html Inline HTML [Element: li]
README.md:38:7 MD033/no-inline-html Inline HTML [Element: a]
README.md:40:5 MD033/no-inline-html Inline HTML [Element: li]
README.md:41:7 MD033/no-inline-html Inline HTML [Element: a]
README.md:43:5 MD033/no-inline-html Inline HTML [Element: li]
README.md:44:7 MD033/no-inline-html Inline HTML [Element: a]
README.md:45:7 MD033/no-inline-html Inline HTML [Element: ul]
README.md:46:9 MD033/no-inline-html Inline HTML [Element: li]
README.md:46:13 MD033/no-inline-html Inline HTML [Element: a]
README.md:49:5 MD033/no-inline-html Inline HTML [Element: li]
README.md:49:9 MD033/no-inline-html Inline HTML [Element: a]
README.md:57:1 MD033/no-inline-html Inline HTML [Element: p]
README.md:57:19 MD033/no-inline-html Inline HTML [Element: a]
README.md:83:1 MD033/no-inline-html Inline HTML [Element: p]
README.md:83:19 MD033/no-inline-html Inline HTML [Element: a]
README.md:87:1 MD033/no-inline-html Inline HTML [Element: p]
README.md:87:19 MD033/no-inline-html Inline HTML [Element: a]
README.md:97:1 MD033/no-inline-html Inline HTML [Element: p]
README.md:97:19 MD033/no-inline-html Inline HTML [Element: a]
README.md:103:1 MD033/no-inline-html Inline HTML [Element: p]
README.md:103:19 MD033/no-inline-html Inline HTML [Element: a]

@stephanbcbauer
Copy link
Member

Right now there are still some errors in the files. @stephanbcbauer how should we handle the following linting errors?

docs/samples/Transfer Data.md:9:65 MD051/link-fragments Link fragments should be valid [Context: "[chapter 1](#1--optional--local-setup)"]
docs/samples/Transfer Data.md:13:4 MD051/link-fragments Link fragments should be valid [Context: "[(optional) Local Setup](#1--optional--local-setup)"]
README.md:1:1 MD033/no-inline-html Inline HTML [Element: a]
README.md:1 MD041/first-line-heading/first-line-h1 First line in a file should be a top-level heading [Context: "<a name="readme-top"></a>"]
README.md:10:1 MD033/no-inline-html Inline HTML [Element: br]
README.md:11:1 MD033/no-inline-html Inline HTML [Element: div]
README.md:12:3 MD033/no-inline-html Inline HTML [Element: a]
README.md:13:5 MD033/no-inline-html Inline HTML [Element: img]
README.md:16:3 MD033/no-inline-html Inline HTML [Element: h3]
README.md:17:3 MD033/no-inline-html Inline HTML [Element: h4]
README.md:19:3 MD033/no-inline-html Inline HTML [Element: p]
README.md:21:5 MD033/no-inline-html Inline HTML [Element: br]
README.md:22:5 MD033/no-inline-html Inline HTML [Element: a]
README.md:22:79 MD033/no-inline-html Inline HTML [Element: strong]
README.md:23:5 MD033/no-inline-html Inline HTML [Element: br]
README.md:24:5 MD033/no-inline-html Inline HTML [Element: br]
README.md:25:5 MD033/no-inline-html Inline HTML [Element: a]
README.md:27:5 MD033/no-inline-html Inline HTML [Element: a]
README.md:29:5 MD033/no-inline-html Inline HTML [Element: a]
README.md:34:1 MD033/no-inline-html Inline HTML [Element: details]
README.md:35:3 MD033/no-inline-html Inline HTML [Element: summary]
README.md:36:3 MD033/no-inline-html Inline HTML [Element: ol]
README.md:37:5 MD033/no-inline-html Inline HTML [Element: li]
README.md:38:7 MD033/no-inline-html Inline HTML [Element: a]
README.md:40:5 MD033/no-inline-html Inline HTML [Element: li]
README.md:41:7 MD033/no-inline-html Inline HTML [Element: a]
README.md:43:5 MD033/no-inline-html Inline HTML [Element: li]
README.md:44:7 MD033/no-inline-html Inline HTML [Element: a]
README.md:45:7 MD033/no-inline-html Inline HTML [Element: ul]
README.md:46:9 MD033/no-inline-html Inline HTML [Element: li]
README.md:46:13 MD033/no-inline-html Inline HTML [Element: a]
README.md:49:5 MD033/no-inline-html Inline HTML [Element: li]
README.md:49:9 MD033/no-inline-html Inline HTML [Element: a]
README.md:57:1 MD033/no-inline-html Inline HTML [Element: p]
README.md:57:19 MD033/no-inline-html Inline HTML [Element: a]
README.md:83:1 MD033/no-inline-html Inline HTML [Element: p]
README.md:83:19 MD033/no-inline-html Inline HTML [Element: a]
README.md:87:1 MD033/no-inline-html Inline HTML [Element: p]
README.md:87:19 MD033/no-inline-html Inline HTML [Element: a]
README.md:97:1 MD033/no-inline-html Inline HTML [Element: p]
README.md:97:19 MD033/no-inline-html Inline HTML [Element: a]
README.md:103:1 MD033/no-inline-html Inline HTML [Element: p]
README.md:103:19 MD033/no-inline-html Inline HTML [Element: a]

@florianrusch-zf when i did the changes/fixes i also fixed these error manual.

@florianrusch-zf
Copy link
Contributor Author

All linting errors are fixed:

$ markdownlint-cli2-config .markdownlint.yaml "**/*.md"
markdownlint-cli2-config v0.6.0 (markdownlint v0.27.0)
Finding: **/*.md
Linting: 58 file(s)
Summary: 0 error(s)

@florianrusch-zf florianrusch-zf marked this pull request as ready for review March 23, 2023 05:47
CODE_OF_CONDUCT.md Outdated Show resolved Hide resolved
Copy link
Member

@stephanbcbauer stephanbcbauer left a comment

Choose a reason for hiding this comment

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

Hey @florianrusch-zf as I remember you also added a link to the installation manual in the root readme.md.

Right now I can't find it. Can we add it? Or maybe with a separate PR?

@florianrusch-zf
Copy link
Contributor Author

florianrusch-zf commented Mar 24, 2023

@stephanbcbauer I haven't add this to the PR as the INSTALL.md is not finished yet. As you maybe remember, we discussed this week that the "All-in-one Deployment" was removed and we need to change this.

@florianrusch-zf florianrusch-zf changed the title refactor: Lint all *.md files and replace product-edc with tractusx-edc docs: Lint all *.md files and replace product-edc with tractusx-edc Mar 27, 2023
@florianrusch-zf florianrusch-zf added the documentation Improvements or additions to documentation label Mar 27, 2023
Copy link
Member

@stephanbcbauer stephanbcbauer left a comment

Choose a reason for hiding this comment

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

LGTM

@florianrusch-zf florianrusch-zf force-pushed the refactor/lint-markdown-files branch from 1c026a0 to b2a0156 Compare March 28, 2023 04:48
@florianrusch-zf
Copy link
Contributor Author

I rebased the branch on the current develop

@SebastianBezold SebastianBezold merged commit 1640d14 into eclipse-tractusx:develop Mar 28, 2023
@florianrusch-zf florianrusch-zf deleted the refactor/lint-markdown-files branch April 5, 2023 06:59
paullatzelsperger pushed a commit to paullatzelsperger/tractusx-edc that referenced this pull request May 4, 2023
…/lint-markdown-files

docs: Lint all *.md files and replace product-edc with tractusx-edc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants