Skip to content

doc: add pymarkdown tool#91

Merged
dbkinder merged 1 commit intoopea-project:mainfrom
dbkinder:add-pymarkdown
Sep 6, 2024
Merged

doc: add pymarkdown tool#91
dbkinder merged 1 commit intoopea-project:mainfrom
dbkinder:add-pymarkdown

Conversation

@dbkinder
Copy link
Contributor

@dbkinder dbkinder commented Sep 5, 2024

pymarkdown provides checks for properly written markdown files. It's quite configurable and checks for many items we choose not to enforce, such as 80 character line length. It can be configured so a CI system could do checks on markdown files being submitted in a PR, but for now, I'm just including the tool in the list of packages installed from PyPI and a script to call it (scripts/checkmd.sh) with some settings that limit what checks we're interested in (for now).

pymarkdown provides checks for properly written markdown files.
It's quite configurable and checks for many items we choose not to
enforce, such as 80 character line length.  It can be configured so a CI
system could do checks on markdown files being submitted in a PR, but
for now, I'm just including the tool in the the list of packages
installed from pypi and a script to call it (checkmd.sh) with some
settings that limit what checks we're interested in (for now).

Signed-off-by: David B. Kinder <david.b.kinder@intel.com>
Copy link
Collaborator

@mkbhanda mkbhanda left a comment

Choose a reason for hiding this comment

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

LGTM

@dbkinder dbkinder merged commit 0eb4ab0 into opea-project:main Sep 6, 2024
@dbkinder dbkinder deleted the add-pymarkdown branch September 25, 2024 01:38
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.

3 participants