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 None access exception #344

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

tyru
Copy link
Collaborator

@tyru tyru commented Jan 11, 2020

When empty entry in config like this:

policies:

vint always raises an exception.

AttributeError: 'NoneType' object has no attribute 'keys'

When empty entry in config like this:

```
policies:
```

vint always raises an exception.

```
AttributeError: 'NoneType' object has no attribute 'keys'
```
@tyru
Copy link
Collaborator Author

tyru commented Jan 11, 2020

CI failed but I don't know how to fix this...

@blueyed
Copy link
Member

blueyed commented Jan 11, 2020

Thanks, remove support for Python 3.4, and rebased it.
Might be good to have a test, and/or better error message in this case.

@coveralls
Copy link

coveralls commented Jan 11, 2020

Coverage Status

Coverage decreased (-0.1%) to 96.28% when pulling 7758955 on tyru:fix-none-empty-entry into 427de13 on Kuniwak:master.

@codecov-io
Copy link

codecov-io commented Jan 11, 2020

Codecov Report

Merging #344 into master will decrease coverage by 0.14%.
The diff coverage is 68.18%.

@@           Coverage Diff            @@
##           master   #344      +/-   ##
========================================
- Coverage   94.14%    94%   -0.15%     
========================================
  Files         133    133              
  Lines        4152   4167      +15     
  Branches      417    418       +1     
========================================
+ Hits         3909   3917       +8     
- Misses        150    156       +6     
- Partials       93     94       +1

@tyru
Copy link
Collaborator Author

tyru commented Jan 12, 2020

@blueyed sorry, I have no enough time to do it.
if you feel 7758955 is good, please merge it,
or wait until for a while... or you can implement it :)

@blueyed blueyed added the bug label Jan 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants