Skip to content

ci: add flake8, black, and pytest-cov - #177

Merged
SharanyaAchanta merged 4 commits into
SharanyaAchanta:mainfrom
Aspect022:feature/ci-enhancements
Mar 3, 2026
Merged

ci: add flake8, black, and pytest-cov#177
SharanyaAchanta merged 4 commits into
SharanyaAchanta:mainfrom
Aspect022:feature/ci-enhancements

Conversation

@Aspect022

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Rationale for this change

The current CI workflow relies only on standard test execution via pytest. To ensure consistent code style among contributors and prevent unnoticed drops in test coverage, automated checks for formatting, linting, and coverage tracking needed to be integrated into the GitHub Actions pipeline.

What changes are included in this PR?

  • Dependencies: Added flake8, black, and pytest-cov to the requirements.txt under the Testing section.
  • CI Pipeline (.github/workflows/lextransition-ci.yml):
    • Added a black check to enforce consistent code formatting (black --check .).
    • Added a flake8 execution step to surface syntax errors and undefined names as well as track complexity.
    • Updated the existing pytest command to utilize pytest-cov to output both a terminal and XML coverage report for PRs.

(Note: We deliberately kept the threshold flags off for coverage right now to prevent immediately failing the CI, allowing the team to establish the baseline and address currently breaking tests sequentially.)

Are these changes tested?

Yes. The workflow updates were validated to ensure that GitHub actions will trigger the new linting, formatting, and coverage tools correctly on incoming PRs and pushes.

Are there any user-facing changes?

No user-facing application changes. These modifications are strictly related to internal CI/CD infrastructure and developer tooling.

@github-actions

Copy link
Copy Markdown

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@centiceron centiceron added the On hold Temporary "on hold" status for maintainence label Feb 27, 2026
@SharanyaAchanta

Copy link
Copy Markdown
Owner

Hey, thanks for the PRs! I was caught up with other merges yesterday. Since today is the final day, I’m wrapping things up. I see some checks are failing now—if you can resolve them quickly, let me know. If not, no worries at all, thanks for your contribution!

@SharanyaAchanta

Copy link
Copy Markdown
Owner

Hey @Aspect022 Please resolve the merge conflicts too

@Aspect022

Copy link
Copy Markdown
Contributor Author

yes i am solving them rn

@Aspect022
Aspect022 force-pushed the feature/ci-enhancements branch from 6034e42 to 88e73ec Compare March 3, 2026 06:35
@Aspect022

Copy link
Copy Markdown
Contributor Author

@SharanyaAchanta i have done it can you chcek noww

@SharanyaAchanta

Copy link
Copy Markdown
Owner

@Aspect022 Conflicts are resolved now but one check is failing

@Aspect022

Copy link
Copy Markdown
Contributor Author

@SharanyaAchanta all done all checks have passsed!!

@SharanyaAchanta
SharanyaAchanta merged commit 796ada5 into SharanyaAchanta:main Mar 3, 2026
1 check passed
@Aspect022
Aspect022 deleted the feature/ci-enhancements branch March 3, 2026 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

med On hold Temporary "on hold" status for maintainence OSCG26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Linting (flake8/black) and Test Coverage to CI Pipeline

3 participants