[IMP] oca_pre_commit_hooks: Add --list-msgs option (#44)

OCA / odoo-pre-commit-hooks

Linters of Odoo addons that complement pylint-odoo

GNU Affero General Public License v3.0

9 stars 11 forks source link

Closed antonag32 closed 1 year ago

antonag32 commented 1 year ago

Fixes #44 Calling entry points with the argument --list-msgs prints out a list of all enabled messages.

antonag32 commented 1 year ago

Seems to be failing due to missing unit tests (decreased coverage). Working on it as I want to find the cleanest way to test this.

moylop260 commented 1 year ago

The PR doesn't have issue related
The list-msgs should be similar to update_readme method where all checks are extracted from docstring of the methods using regex
and should show all them (without processing enabled/disabled)
and return early similar to --help
inspired from pylint using --list-msgs

antonag32 commented 1 year ago

Got it, got caught up in other coding shenanigans regarding classes/reusing code last night and was derailed from the main point.

I think the PR is good now. However should we even write unit tests for this feature?