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

Remove lazy_static #52

Closed
wants to merge 1 commit into from

Conversation

csking101
Copy link
Contributor

#42
Removed lazy_static and implemented OnceLock

@csking101 csking101 force-pushed the remove_lazy_static branch from 256563a to 3b3f655 Compare March 10, 2024 06:32
Copy link
Collaborator

@gabhijit gabhijit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Good to me! Can you rebase and squash this into a single commit - so that I can merge this directly!

@csking101
Copy link
Contributor Author

I'd rebased already it in the force push, do I need to do it once again to squash into a single commit?

@gabhijit
Copy link
Collaborator

gabhijit commented Mar 10, 2024

@csking101 : What I generally do is - if there are more than one commits - Just squash into one logical commit, so that the history looks okay. I can do it for you as well. But if you do it, I will just merge it.

Signed-off-by: csking101 <chinmayasahu101@gmail.com>

Removed lazy_static

Signed-off-by: csking101 <chinmayasahu101@gmail.com>

Some cosmetic changes

Signed-off-by: csking101 <chinmayasahu101@gmail.com>
@csking101 csking101 force-pushed the remove_lazy_static branch from 3b3f655 to 4e4ac6f Compare March 10, 2024 16:24
@csking101
Copy link
Contributor Author

I have tried squashing it now, could you please check sir.

@gabhijit
Copy link
Collaborator

@csking101 : I have made minor edits to the git commit message and then merged that. This functionality is now available in the main branch.

Thank you for your contribution.

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