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

Universe: remove Bazel files #2794

Merged
merged 2 commits into from
Jul 27, 2021
Merged

Universe: remove Bazel files #2794

merged 2 commits into from
Jul 27, 2021

Conversation

mrtnzlml
Copy link
Member

The reasoning is fairly simple: nobody is actively working on
integrating with Bazel and there are no plans/capacity to do so. It
might be because Bazel is quite complex, and it's hard to set up it in a
way we need it to be. Moreover, it's blocking other efforts like this
one for example: #2154 (Yarn 2).

Note: this doesn't mean that there is no need for something like Bazel.
However, it doesn't have to be Bazel (preferably something simpler that
does the job well enough).

The reasoning is fairly simple: nobody is actively working on
integrating with Bazel and there are no plans/capacity to do so. It
might be because Bazel is quite complex, and it's hard to set up it in a
way we need it to be. Moreover, it's blocking other efforts like this
one for example: #2154 (Yarn 2).

Note: this doesn't mean that there is no need for something like Bazel.
However, it doesn't have to be Bazel (preferably something simpler that
does the job well enough).
@mrtnzlml mrtnzlml requested a review from a team as a code owner July 26, 2021 01:44
@mrtnzlml
Copy link
Member Author

This is actually the first commit in a series of cleanup commits. Specifically, Shipit needs some cleaning, however, I think it should happen after this one.

@kodiakhq kodiakhq bot merged commit 4806cd4 into master Jul 27, 2021
@kodiakhq kodiakhq bot deleted the remove_bazel_files branch July 27, 2021 21:07
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