search

OCA / pylint-odoo

Odoo plugin for Pylint
http://www.pylint.org
143 stars 168 forks source link

[REF] resource-not-exist: Consider the manifest's key `images` #447

Closed moylop260 closed 1 year ago

moylop260 commented 1 year ago

images has a local path to image file so we need to consider it from the check resource-not-exists

voronind commented 1 year ago

AFAIK images has non-local path in common case

moylop260 commented 1 year ago

Could you share a module using a non-local path, please?

FYI I just ran locally the following to get all the non-local path: rgrep "images" --include=__manifest__.py |grep http |wc -l Result: 0

Local paths: rgrep "images" --include=__manifest__.py | wc -l 375

Please, share more details

github-actions[bot] commented 1 year ago

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.