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 a potential panic when querying authenticator status #395

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

dyc3
Copy link
Owner

@dyc3 dyc3 commented Jul 3, 2024

No description provided.

Copy link
Owner Author

dyc3 commented Jul 3, 2024

@dyc3 dyc3 changed the title remove an unnecessary unwrap fix a potential panic when querying authenticator status Jul 3, 2024
@dyc3 dyc3 marked this pull request as ready for review July 3, 2024 13:11
@dyc3 dyc3 force-pushed the 07-03-remove_an_unnecessary_unwrap branch from 629e218 to 2c02533 Compare July 3, 2024 13:14
@dyc3 dyc3 changed the base branch from master to 07-03-fix_new_lints July 3, 2024 13:14
@dyc3 dyc3 mentioned this pull request Jul 3, 2024
Copy link
Owner Author

dyc3 commented Jul 17, 2024

Merge activity

  • Jul 16, 8:07 PM EDT: @dyc3 started a stack merge that includes this pull request via Graphite.
  • Jul 16, 8:08 PM EDT: Graphite rebased this pull request as part of a merge.
  • Jul 16, 8:13 PM EDT: @dyc3 merged this pull request with Graphite.

Base automatically changed from 07-03-fix_new_lints to master July 17, 2024 00:07
@dyc3 dyc3 force-pushed the 07-03-remove_an_unnecessary_unwrap branch from 2c02533 to ceec318 Compare July 17, 2024 00:07
@dyc3 dyc3 merged commit dd153a6 into master Jul 17, 2024
3 checks passed
@dyc3 dyc3 deleted the 07-03-remove_an_unnecessary_unwrap branch July 17, 2024 00:13
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.

1 participant