-
Notifications
You must be signed in to change notification settings - Fork 273
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
[PINS] Add Bazel support for external projects #557
Conversation
@qiluo-msft this is related to #547 |
Thanks Qi for the approval, Before merging I think we can remove the tests/fakes/BUILD, and tests/mocks/BUILD files. |
Those files are removed. Just need to rebase after #545 is merged. |
Bazel build support allows swss-common to used in Bazel projects; this doesn’t change/impact the make-based build system Additionally, - Add Bazel test to CI - Update .gitignore Co-authored-by: Robert J. Halstead [email protected] Co-authored-by: Runming Wu [email protected]
/azpw run |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azpw run |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
Thanks for the cleanup Brian. Looks like the tests are passing as well. Qi do you mind approving and merging the changes? |
Thanks Qi! |
Bazel build support allows swss-common to used
in Bazel projects; this doesn’t change/impact the
make-based build system
Submission containing materials of a third party:
Copyright Google LLC; Licensed under Apache 2.0
Co-authored-by: Akarsh Gupta [email protected]
Co-authored-by: Ashish Singh [email protected]
Co-authored-by: Jay Hu [email protected]
Co-authored-by: Manali Kumar [email protected]
Co-authored-by: Richard Yu [email protected]
Co-authored-by: Robert J. Halstead [email protected]
Co-authored-by: Runming Wu [email protected]
Co-authored-by: Srikishen Pondicherry Shanmugam [email protected]
Co-authored-by: Vivek Ramamoorthy [email protected]
Co-authored-by: Yilan Ji [email protected]
TODO
ETA: this week in a separate PR)