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

[FIX] missing-newline-extrafiles: Fix 'IOError: [Errno 22] Invalid argument' for files of size 2 #37

Merged
merged 1 commit into from
May 31, 2016

Conversation

moylop260
Copy link
Collaborator

Fix the traceback error:

Traceback (most recent call last):
  File "./pylint_odoo/test/main.py", line 86, in test_20_expected_errors
    pylint_res = self.run_pylint(self.paths_modules)
  File "./pylint_odoo/test/main.py", line 75, in run_pylint
    return Run(self.default_options + extra_params + paths, exit=False)
  File "./pylint/lint.py", line 1332, in __init__
    linter.check(args)
  File "./pylint/lint.py", line 747, in check
    self._do_check(files_or_modules)
  File "./pylint/lint.py", line 869, in _do_check
    self.check_astroid_module(ast_node, walker, rawcheckers, tokencheckers)
  File "./pylint/lint.py", line 946, in check_astroid_module
    walker.walk(ast_node)
  File "./pylint/utils.py", line 871, in walk
    cb(astroid)
  File "./pylint_odoo/checkers/modules_odoo.py", line 108, in visit_module
    self.wrapper_visit_module(node)
  File "./pylint_odoo/misc.py", line 102, in wrapper_visit_module
    if not check_method():
  File "./pylint_odoo/checkers/modules_odoo.py", line 336, in _check_missing_newline_extrafiles
    fp.seek(-2, os.SEEK_END)
IOError: [Errno 22] Invalid argument

@moylop260 moylop260 self-assigned this May 28, 2016
@moylop260 moylop260 changed the title [FIX] missing-newline-extrafiles: Fix 'IOError: [Errno 22] Invalid argument' by files of size 2 [FIX] missing-newline-extrafiles: Fix 'IOError: [Errno 22] Invalid argument' for files of size 2 May 28, 2016
@coveralls
Copy link

coveralls commented May 28, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling c11735d on vauxoo-dev:master-fix-inval-arg-moy into fed468c on OCA:master.

@pedrobaeza
Copy link
Member

👍

2 similar comments
@dreispt
Copy link
Sponsor Member

dreispt commented May 30, 2016

👍

@luiseevaquer
Copy link

👍

@pedrobaeza pedrobaeza merged commit fd91cb4 into OCA:master May 31, 2016
@moylop260 moylop260 deleted the master-fix-inval-arg-moy branch June 1, 2016 15:17
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.

None yet

5 participants