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(wl2k): Fix exception handling #139

Merged
merged 1 commit into from Apr 1, 2022
Merged

Conversation

wb8tyw
Copy link
Owner

@wb8tyw wb8tyw commented Apr 1, 2022

d_rats/wl2k.py:
Catch specific exceptions.
Raise module specific named exceptions.
Remove some unused code.

d_rats/wl2k.py:
  Catch specific exceptions.
  Raise module specific named exceptions.
  Remove some unused code.
@wb8tyw
Copy link
Owner Author

wb8tyw commented Apr 1, 2022

Still no functional changes to the code or python 3 fixes.

This just fixes the exception handling to comply with current python conventions and cleans up some more pylint messages.

Next step is to mock up a Winlink server for unit testing the code.

@wb8tyw wb8tyw merged commit 88f36a3 into python3_tyw Apr 1, 2022
@wb8tyw wb8tyw deleted the wb8tyw_w2lk_except branch April 1, 2022 23:58
wb8tyw added a commit that referenced this pull request Sep 5, 2022
d_rats/wl2k.py:
  Catch specific exceptions.
  Raise module specific named exceptions.
  Remove some unused code.

Co-authored-by: John E. Malmberg <wb8tyw@gmail.com>
wb8tyw added a commit that referenced this pull request Sep 6, 2022
d_rats/wl2k.py:
  Catch specific exceptions.
  Raise module specific named exceptions.
  Remove some unused code.

Co-authored-by: John E. Malmberg <wb8tyw@gmail.com>
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