False warning C7902(missing-readme) on some repo

[legalsylvain]

False warning C7902(missing-readme) on some repo

[sbidoul]

Does it make sense to use pylint-odoo for this project? I'd think flake8 is sufficient. Or you can disable messages in .pylintrc?

But overall I think the oca addons repo template does not apply to openupgrade, as it is meant to be used with... addons repos :)

[legalsylvain]

But overall I think the oca addons repo template does not apply to openupgrade, as it is meant to be used with... addons repos :)

But that's now addons repo. See :

• https://pypi.org/project/odoo-addon-openupgrade-framework/
• https://pypi.org/project/odoo-addon-openupgrade-scripts/

Does it make sense to use pylint-odoo for this project? I'd think flake8 is sufficient. Or you can disable messages in .pylintrc?

Some checks maybe doesn't make sense or will be useless, but framework module contains code that can be "pylinted" and migrations scripts contains python files that can be also checked. relative-import, sql-injection, ...
In fact, for the time being, I just disabled one check here :
https://github.com/OCA/OpenUpgrade/blob/16.0/.pylintrc#L72 because there are a lot of xml files. (noupdate_changes.xml) that are not imported in the manifest. (file-not-used)

Well, my question is why pylint think that a subfolder is a module.

(If not found, indeed I can disable missing-readme, but I think that something is wrong somewhere).

[sbidoul]

But that's now addons repo

Ah, ok, yes I remember that discussion.

Well, my question is why pylint think that a subfolder is a module.

That's probably a question for the pylint-odoo project then.