search

Vauxoo / pylint-odoo

1 stars 4 forks source link

[IMP] manifest-required-author: Allow multiple possible required authors #158

Closed luisg123v closed 6 years ago

luisg123v commented 6 years ago

There are cases when the required autor for modules may be one among several ones, e.g. the author is required, but it is not always the same. For cases like those, it is useful to support multiple values for this lint.

This commit improves the lint so it accepts multiple values separated by commas.

Test cases are provided to cover one and multiple values passed to the parameter to set required authors.

This is a cherry-pick of https://github.com/OCA/pylint-odoo/commit/8dcb1eb9df1e8623680f20f698d58b6e325a67de

luisg123v commented 6 years ago

@moylop260
Could you merge this, please?

After mergin this, please merge also https://github.com/Vauxoo/pylint-conf/pull/55

moylop260 commented 6 years ago

Could you create a PR to OCA, please?

luisg123v commented 6 years ago

@moylop260 This is a cherry-pick of OCA@8dcb1eb, so there is no need to create a PR to the OCA, it's already merged there.