Disable Pylint checks on __unported__

[yvaucher]

In OCA/connector-magento#75 I encounter pylint check on __unported__ dir I don't think we want to check that directory too.

[dreispt]

That shouldn't be so.
@moylop260 can you check?

[pedrobaeza]

That it's not supposed to happen...

[moylop260]

@yvaucher
Sorry for delay.
I need more information.
Can you give me a build link of travis with problem, please?

[dreispt]

I'm not aware of this being an issue right now. I vote to close.

[pedrobaeza]

We can __unported__ to https://github.com/OCA/maintainer-quality-tools/blob/master/travis/cfg/travis_run_pylint.cfg#L3

[pedrobaeza]

flake8 check already have it: https://github.com/OCA/maintainer-quality-tools/blob/master/travis/cfg/travis_run_flake8.cfg#L6

[moylop260]

@pedrobaeza
Do you have a travis build where pylint check __unported__ folder?
pylint don't check paths if not exists a python package (__init__.py in root path).

[moylop260]

I found next case:
travis build with unported checks

I saw next folder structure in __unported__ folder:

I did not know that we added __init__.py file to __unported__ folder.
Is required this structure for you?

[moylop260]

I saw this change of @guewen:
OCA/connector-magento@e8ad2d8#diff-68a29d45b5dc96e3451c43107948f08a

I detect next case:

File renamed without changes
customize_example/__init__.py → __unported__/__init__.py

IMHO this should be:

File renamed without changes
customize_example/__init__.py → __unported__/customize_example/__init__.py

[moylop260]

IMHO this is a expected error of pylint.
I will close this issue.
Feel free to reopen if you consider it necessary.

[guewen]

IMHO this should be:

Of course, it should have been that. It will be corrected once my PR OCA/connector-magento#85 is merged.