Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REF] Use real file path in no-py checks #62

Merged
merged 4 commits into from
Jul 22, 2016

Conversation

moylop260
Copy link

@moylop260 moylop260 commented Jul 21, 2016

Change from
pylint_odoo/test_repo/broken_module/__init__:1: [W7910(wrong-tabs-instead-of-spaces), ] model_view_odoo.xml:23 Use wrong tabs indentation instead of four spaces
to
pylint_odoo/test_repo/broken_module/model_view_odoo.xml:23: [W7910(wrong-tabs-instead-of-spaces), ] Use wrong tabs indentation instead of four spaces

@moylop260 moylop260 self-assigned this Jul 21, 2016
@moylop260 moylop260 changed the title WIP Use no-py real file name [REF] Use real file path in no-py checks Jul 21, 2016
@moylop260 moylop260 merged commit 7ab9346 into Vauxoo:master Jul 22, 2016
@moylop260 moylop260 deleted the master-vx-xml-lineno-moy branch July 22, 2016 19:31
moylop260 added a commit to vauxoo-dev/pylint-odoo that referenced this pull request Aug 20, 2016
moylop260 added a commit to vauxoo-dev/pylint-odoo that referenced this pull request Sep 14, 2016
… line number in non-py files (Vauxoo#62)

[REF] global refactoring: better message output and use real file and line number in non-py files

Change from `pylint_odoo/test_repo/broken_module/__init__:1: [W7910(wrong-tabs-instead-of-spaces), ]  model_view_odoo.xml:23 Use wrong tabs indentation instead of four spaces`
to `pylint_odoo/test_repo/broken_module/model_view_odoo.xml:23: [W7910(wrong-tabs-instead-of-spaces), ]  Use wrong tabs indentation instead of four spaces`
moylop260 added a commit that referenced this pull request Dec 9, 2016
* [REF] global refactoring: better message output and use real file and line number in non-py files (#62)

[REF] global refactoring: better message output and use real file and line number in non-py files

Change from `pylint_odoo/test_repo/broken_module/__init__:1: [W7910(wrong-tabs-instead-of-spaces), ]  model_view_odoo.xml:23 Use wrong tabs indentation instead of four spaces`
to `pylint_odoo/test_repo/broken_module/model_view_odoo.xml:23: [W7910(wrong-tabs-instead-of-spaces), ]  Use wrong tabs indentation instead of four spaces`

* [FIX] rst-syntax-error: Skip unknown directives

* [FIX] rst-syntax-error: Skip unknown roles

* [ADD] missing-import-error, missing-manifest-dependency

* [REF] README: Update messages list

Signed-off-by: Moisés López <moylop260@vauxoo.com>

* Adding isort dependency (#70)

* [REF] requirements: Update developer version of pylint and astroid

* [ADD] Support for 10.0 manifest name

* [REF] missing-import-error: Skip test file
since these files are loaded only when running tests and in such a case your module and their external dependencies are installed.

* [IMP] manifest-version-format: Add valid_odoo_versions parameter to force a valid version of odoo in the manifest version

* [FIX] missing-import-error: Updating libraries used from requirements.txt but not imported or nested imported from odoo

* [FIX] manifest-version-format: Support -e manifest-version-format only

* [FIX] manifest-version-format: Fix regex to use explicit dot instead of any char

* [FIX] Whitelist `anybox.testing.openerp`
* Add `anybox.testing.openerp` - Fixes #81

* [ADD] missing-return
If you use call a `super` method then you will need return the original
value.
If you want overwrite a original method then you need add documentation
of why and add a `pylint: disable=missing-return`

* [REF] attribute-deprecated: Deprecate length class attribute (#86)

* [REF] attribute-deprecated: Deprecate length class attribute
Related issue: OCA/maintainer-quality-tools#385 and odoo/odoo#13780

* [REF] .travis.yml: Disable F999

Disable the error: "dictionary key 'name' repeated with  different value"
This error is duplicated from pylint.

* [FIX] method-NAME: Fix case compute=None
Fix OCA#88
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant