Closed zkoppert closed 3 months ago
Ran lint locally and got cleanowners.py:145:58: E0601: Using variable 'codeowners_file_contents_new' before assignment (used-before-assignment)
hence the change to default codeowners_file_contents_new
to None
in my commit.
Looks good! Thanks for adding @jmeridth. Leaving this pr open for @sutterj to take a look at.
Pull Request
fixes #11
Proposed Changes
This pull request introduces a new feature that generates a report of users removed from the
CODEOWNERS
files across the repos that were affected. The report is written in Markdown format and can be optionally created as an issue in the repository. TheISSUE_REPORT
environment variable controls this feature. TheREADME.md
file has been updated to explain this new feature and how to use it.I also added/modified tests to cover the new functionality.
Readiness Checklist
Author/Contributor
make lint
and fix any issues that you have introducedmake test
and ensure you have test coverage for the lines you are introducing@jeffrey-luszcz
Reviewer
fix
,documentation
,enhancement
,infrastructure
,maintenance
orbreaking