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

misc git cleanup by aaronliu0130 · Pull Request #340 · cpplint/cpplint · GitHub

misc git cleanup - #340

Merged
cclauss merged 3 commits into
developfrom
misgit
Mar 16, 2025
Merged

misc git cleanup#340
cclauss merged 3 commits into
developfrom
misgit

Conversation

aaronliu0130 commented Mar 16, 2025
edited
Loading

Copy link
Copy Markdown
Member

Remove my pathetic attempt at a debug print from unit tests
Ignore reformatting commit from git blame
gitignore pycharm and uv.lock
evidently i was not feeling well

Remove my pathetic attempt at a debug print from unit tests
Ignore reformatting commit from git blame
gitignore pycharm and uv.lock
aaronliu0130 requested review from cclauss and jayvdb March 16, 2025 21:01
evidently i was not feeling well
Comment thread cpplint_unittest.py Outdated
Comment thread .gitignore Outdated
ENV/
env.bak/
venv.bak/
uv.lock

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

Please read the uv docs. This document is supposed to be committed.

Suggested change
uv.lock

Copy link
Copy Markdown
Member Author

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 don't want to lock my dependencies, though.

cclauss merged commit f44f079 into develop Mar 16, 2025
cclauss deleted the misgit branch March 16, 2025 23:33
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.

2 participants


Back | FazBrowse Home | New Git URL