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

Refactor Code #140

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

saleh-alhaddad
Copy link

@saleh-alhaddad saleh-alhaddad commented Aug 14, 2024

This PR refactors to improve code readability, maintainability, and robustness.

@BertramScharpf
Copy link

I had almost the same ideas. But instead of waiting several years for a merge, I opened my own project. In case you're interested, it's called BertramScharpf/ruby-popt.

@saleh-alhaddad
Copy link
Author

@BertramScharpf let's mention them again as they are busy in most of the time.

cc: @olleolleolle @mdp @shaiguitar

@mdp mdp self-requested a review January 6, 2025 21:40
@mdp
Copy link
Owner

mdp commented Jan 6, 2025

Sorry, unless there's a security issue or a bug I'm pretty hesitant to push updates to this library, it's a library that doesn't change much (and TOTP has largely been deprecated by security conscious orgs), and due to its security role I'm extra cautious.

That being said, it's been a while since it's been updated and I needed to fix a Github workflow and test issue (was failing on >3.3). I don't want to force a rebase on your PR branch, so I'll let you do it, but if you rebase this PR to 'main' it should run the test workflow and verify that everything passes (I ran them on my own and it looks good). Then I'll merge it into main, although I likely won't push a release unless there some additional changes that come in later and require it.

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.

3 participants