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

Bazel support #33

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

Bazel support #33

wants to merge 1 commit into from

Conversation

zaucy
Copy link

@zaucy zaucy commented Mar 28, 2022

Hi! I was wondering if you'd be willing to accept the bazel build system support I created upstream. If there were any issues in the future I'd be willing to contribute further. I also added some files necessary to support the bazel central registry for easily adding your library to others bazel based projects.

My comments below are only questions if you are willing to merge this in. I understand if you're unwiling to introduce an additional build system to the repository.

Comment on lines +2 to +3
name = "com_github_yohhoy_yamc",
version = "1.0.0",
Copy link
Author

Choose a reason for hiding this comment

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

The name here is how this repository would show up to developers.

I don't know what version you would prefer here since I don't see a cut release in the repository. Let me know!

If you have any preferences please let me know or feel free to edit as you wish!

@yohhoy
Copy link
Owner

yohhoy commented Mar 30, 2022

Thank you for big PR to support Bazel building system.
But I didn't use Bazel yet, so I don't understand these changes exactly...

Does this PR include the following purposes?

    1. Build unit test programs with GTest on bazel
    1. Prepare to expose this library(yamc) on bazel registory(?)

@zaucy
Copy link
Author

zaucy commented Apr 1, 2022

Yes to both! I'm happy to answer any questions about bazel and how someone might use yamc with it.

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