-
Notifications
You must be signed in to change notification settings - Fork 235
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
ci(c++): Implement C++ CI scripts using python #1303
Conversation
7b1434d
to
5cc874c
Compare
PTAL, thanks. |
Could we add windows CI in this PR? |
Windows CI has not been added yet, windows bazel requires some additional configuration, I am trying it locally based on the current For the current linux and mac os C++ CI, it can work. Maybe we can merge the pr first. |
605f9a3
to
21a572f
Compare
Hi @PragmaTwice, It has been revised according to the comments mentioned. In addition, the local adaptation of Windows C++ CI has been completed in the past two days. After this PR is merged, another PR will be created to implement the final adaptation of Windows C++ CI. |
Signed-off-by: LiangliangSui <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Can this PR be merged? If possible, please help me merge it. I don’t have the permission to merge it. |
Implement C++ CI scripts using python.