-
Notifications
You must be signed in to change notification settings - Fork 813
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
Cmake. Removed unused script. #907
Conversation
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.
It doesn't appear to be used anymore, so removing LGTM. I have no authority in this repo, so I'm not sure if my approval means anything :-)
Build Succeeded 👏 Build Id: dfbaa636-decf-48e4-8d09-af9de09b9017 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
/lgtm /assign @markmandel for approval and merge. |
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
Build Failed 😱 Build Id: 8ad74941-06d3-42ec-a1f9-886927d771bb To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Failed 😱 Build Id: 92ebd363-227c-4802-8e7f-35eb206defcf To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Failed 😱 Build Id: a2bfc15e-c769-4d1f-84e2-cd6f829da44e To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: 65ce3a86-c932-47c2-85dd-0c0c3a219744 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
I forgot to remove
Fetch_gRPC.cmake
file in #803It is not used anymore.
@devjgm could you do a review, please?