Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a markdown formatting tool consisting of a Python script (scripts/format.py) and a helper bash script (scripts/format.sh) to automate formatting using mdformat. It also updates .gitignore to exclude the virtual environment directory and applies the formatter to several documentation files, wrapping text to 80 characters and standardizing formatting. The review feedback identifies an opportunity to optimize scripts/format.sh by checking for the existence of executables inside the virtual environment and avoiding redundant pip install calls on every execution, which improves both robustness and performance.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
philschmid
left a comment
There was a problem hiding this comment.
We should noy change the external format. Each linebreaks adds at least 1 token.
Internally skills need to be formatted with
mdformat, and since this project is set up as GH SoT, it needs to be implemented here.Also reformatted all markdown in the repo.