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

[NFC] Add bazel support to cpu_features #222

Merged
merged 2 commits into from
Feb 3, 2022
Merged

Conversation

gchatelet
Copy link
Collaborator

@gchatelet gchatelet commented Feb 1, 2022

Partial fix for #221

@gchatelet gchatelet requested a review from Mizux February 1, 2022 15:39
@gchatelet
Copy link
Collaborator Author

tested with bazelisk-linux-amd64 test ...

WORKSPACE Show resolved Hide resolved
@toor1245
Copy link
Contributor

toor1245 commented Feb 1, 2022

Do we need to provide "Build with Bazel" documentation or will it be a separate patch?

@gchatelet
Copy link
Collaborator Author

@toor1245 good point ! Let's do it as a second step though, I suspect there will be a few iterations before everything works as correctly. We also need to setup continuous integration.

@gchatelet gchatelet merged commit e38dc6d into main Feb 3, 2022
@gchatelet gchatelet deleted the add_bazel_support branch February 3, 2022 12:56
@gchatelet gchatelet added this to the v0.7.0 milestone Mar 8, 2022
@gchatelet gchatelet added the enhancement New feature or request label Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants