FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

README.rst: Add instructions for pre-commit by cclauss · Pull Request #320 · cpplint/cpplint · GitHub

README.rst: Add instructions for pre-commit - #320

Merged
cclauss merged 4 commits into
developfrom
README_pre-commit_pipx_uvx
Mar 6, 2025
Merged

README.rst: Add instructions for pre-commit#320
cclauss merged 4 commits into
developfrom
README_pre-commit_pipx_uvx

Conversation

cclauss commented Mar 4, 2025
edited
Loading

Copy link
Copy Markdown
Member

aaronliu0130 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I've always found information on how to use a project with another project weird when one can very easily figure out how to do so. Such information also potentially opens us up to frivolous PRs asking to add also-easily-figured-out information on how to use this with X project, e.g. "add usage with pipsi", "add autorun command for sublime text".

Comment thread README.rst Outdated
Comment thread README.rst
Co-authored-by: Aaron Liu <aaronliu0130@gmail.com>
cclauss requested a review from aaronliu0130 March 6, 2025 22:54
cclauss changed the title README.rst: Add instructions for pre-commit, pipx run, and uvx README.rst: Add instructions for pre-commit Mar 6, 2025
cclauss merged commit 181d47f into develop Mar 6, 2025
cclauss deleted the README_pre-commit_pipx_uvx branch March 6, 2025 23:27
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL