Opening this as a PR in case of adverse side effects before merging. Will merge in 1-2 days after some additional testing.
Previously AuthenticationError being raised would go unhandled, because run calls create_task(connect) and ignored the output/error. This also removes a wrapping error when calling start with a bad token.
This fixes #344
Checklist
[x] If code changes were made then they have been tested.
[ ] I have updated the documentation to reflect the changes.
[ ] I have updated the changelog with a quick recap of my changes.
[x] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)
Pull request summary
Opening this as a PR in case of adverse side effects before merging. Will merge in 1-2 days after some additional testing.
Previously AuthenticationError being raised would go unhandled, because run calls
create_task(connect)and ignored the output/error. This also removes a wrapping error when calling start with a bad token. This fixes #344Checklist