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

Undefined name: Typo access_addres --> access_address (add a trailing s) #2

Merged
merged 2 commits into from
Aug 13, 2018

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Aug 13, 2018

flake8 testing of https://github.com/virtualabs/btlejack on Python 3.7.0

$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics

./btlejack/session.py:72:12: F821 undefined name 'access_addres'
        if access_addres in self.connections:
           ^
./btlejack/link.py:190:20: F821 undefined name 'RecoverCrcInitCommand'
        self.write(RecoverCrcInitCommand(access_address, channel_map, hop_interval))
                   ^
./btlejack/link.py:191:26: F821 undefined name 'RecoverResponse'
        self.wait_packet(RecoverResponse)
                         ^
3     F821 undefined name 'RecoverCrcInitCommand'
3

… __s__)

[flake8](http://flake8.pycqa.org) testing of https://github.com/virtualabs/btlejack on Python 3.7.0

$ __flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics__
```
./btlejack/session.py:72:12: F821 undefined name 'access_addres'
        if access_addres in self.connections:
           ^
./btlejack/link.py:190:20: F821 undefined name 'RecoverCrcInitCommand'
        self.write(RecoverCrcInitCommand(access_address, channel_map, hop_interval))
                   ^
./btlejack/link.py:191:26: F821 undefined name 'RecoverResponse'
        self.wait_packet(RecoverResponse)
                         ^
3     F821 undefined name 'RecoverCrcInitCommand'
3
```
@cclauss cclauss changed the title Undefined name: Typo access_addres --> access_address (add a trailing… Undefined name: Typo access_addres --> access_address (add a trailing s) Aug 13, 2018
Also, use () instead of backslash because one whitespace character to the right of the backslash breaks the script because of a change that is invisible to the reader.
@virtualabs virtualabs merged commit 7d3cebc into virtualabs:master Aug 13, 2018
@virtualabs
Copy link
Owner

Good eye, thanks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants