Skip to content

Add more linting #646

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Add more linting #646

wants to merge 15 commits into from

Conversation

CarrotManMatt
Copy link
Member

Adds flake8-carrot & yamllint linting

@CarrotManMatt CarrotManMatt self-assigned this Oct 16, 2025
@CarrotManMatt CarrotManMatt added the deployment Changes to the deployment or CI/CD configuration label Oct 16, 2025
@CarrotManMatt CarrotManMatt force-pushed the add-flake8-carrot-linting branch from 8a07a16 to 8ec5502 Compare October 16, 2025 12:50
@codecov Codecov
Copy link

codecov bot commented Oct 16, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
utils/tex_bot.py 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@CarrotManMatt CarrotManMatt marked this pull request as ready for review October 16, 2025 14:26
@CarrotManMatt CarrotManMatt added the sync Request bots to automatically keep this PR up to date with it's base branch label Oct 16, 2025
MattyTheHacker
MattyTheHacker requested changes Oct 20, 2025
View reviewed changes
@CarrotManMatt
Update check-build-deploy.yaml
ee76c45 
Signed-off-by: Matt Norton <[email protected]>
@CarrotManMatt
Update utils.py
abd0445 
Signed-off-by: Matt Norton <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MattyTheHacker MattyTheHacker Awaiting requested review from MattyTheHacker MattyTheHacker is a code owner

Requested changes must be addressed to merge this pull request.

Assignees

@CarrotManMatt CarrotManMatt

Labels

deployment Changes to the deployment or CI/CD configuration sync Request bots to automatically keep this PR up to date with it's base branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@CarrotManMatt @MattyTheHacker