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: spelling of dbus_fast.auth.AuthAnnonymous to dbus_fast.auth.AuthAnonymous #220

Merged
merged 3 commits into from
Aug 5, 2023

Conversation

jimduchek
Copy link
Contributor

Pretty straightforward. Don't think very many people use the ANONYMOUS auth, but included a line to keep backwards compatibility to not break anybody using the misspelled version.

@codecov
Copy link

codecov bot commented Aug 5, 2023

Codecov Report

Patch coverage: 66.66% and project coverage change: +0.12% 🎉

Comparison is base (5b6cbc5) 82.60% compared to head (4ccf8da) 82.73%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #220      +/-   ##
==========================================
+ Coverage   82.60%   82.73%   +0.12%     
==========================================
  Files          27       27              
  Lines        3220     3221       +1     
  Branches      668      668              
==========================================
+ Hits         2660     2665       +5     
+ Misses        344      342       -2     
+ Partials      216      214       -2     
Files Changed Coverage Δ
src/dbus_fast/__version__.py 0.00% <0.00%> (ø)
src/dbus_fast/auth.py 88.13% <100.00%> (+0.20%) ⬆️

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bdraco
Copy link
Member

bdraco commented Aug 5, 2023

This looks fine. Please add a test to make sure both names work so we do not regress this in the future by accident

@jimduchek
Copy link
Contributor Author

Good call. Will add that now.

@jimduchek
Copy link
Contributor Author

Think that's as good as it's gonna get... the codecov check is failing in a file I didn't touch, so not sure what that's about.

@bdraco
Copy link
Member

bdraco commented Aug 5, 2023

thanks @jimduchek

@bdraco bdraco changed the title 'Annonymous' spelling fix fix: spelling of dbus_fast.auth.AuthAnnonymous to dbus_fast.auth.AuthAnonymous Aug 5, 2023
@bdraco bdraco merged commit 6c2412f into Bluetooth-Devices:main Aug 5, 2023
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.

2 participants