Remove WORKSPACE files - #270
Conversation
There was a problem hiding this comment.
If we do not need to support WORKSPACE files, I'm all for this change!
Keep in mind that CI has not run for whatever reason!
I was hesitant to take this step due to Bazel 7 still supporting WORKSPACE files.
If we merge this don't forget to close: #258.
I think WORKSPACE support might be still important for Bazel 7, @furtib, @Szelethus, please let me know if it is OK to merge since CI checks seem broken. |
8b454b7 to
f7be4ab
Compare
CI checks were affected by GitHub incident (https://www.githubstatus.com/incidents/qcvjkzcs7j74) which is fixed now. |
Why:
Bazel 8 disables the WORKSPACE by default and Bazel 9 removed it.
Our WORKSPACE file is not used FOSS tests were skipped on Bazel 8,
which left yaml-cpp untested there.
What: