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.

Sign up for GitHub

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

Jump to bottom

ci: use visibility crate for bench #50

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

ci: use visibility crate for bench #50

wants to merge 1 commit into from

Conversation

somehowchris
Copy link
Contributor

@somehowchris somehowchris commented Dec 7, 2022 •
edited
Loading

As I got to know the visibility crate, I'm proposing the change to use it.

Alsongside this uses the changes of #46 but using static therefore removing the dependency on lazy_static.

@somehowchris somehowchris force-pushed the feature/use-visibility branch from dae8674 to c6857e9 Compare December 7, 2022 14:08
@somehowchris somehowchris changed the title ci: use crate for bench ci: use visbility crate for bench Dec 8, 2022
@somehowchris somehowchris changed the title ci: use visbility crate for bench ci: use visbility crate for bench Dec 8, 2022
@somehowchris somehowchris changed the title ci: use visbility crate for bench ci: use visibility crate for bench Dec 8, 2022
@somehowchris
Copy link
Contributor Author

@felipenoris hey mate, any chance of getting this project going again?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@somehowchris