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

Build all targets inside a package #72

Closed
A-Sam opened this issue Mar 19, 2019 · 8 comments
Closed

Build all targets inside a package #72

A-Sam opened this issue Mar 19, 2019 · 8 comments
Labels
type: feature request New feature or request

Comments

@A-Sam
Copy link

A-Sam commented Mar 19, 2019

ATM, all targets are being loaded in the view (Bazel build targets). Every time i want to run all the targets inside a package i have to write the 1 liner for loop in bash to build all targets read from $(bazel query //...). It would be good option to add to this view.

@thomasvl
Copy link
Member

fyi - bazel build //..., no need for the query; you can also do bazel build //package:all to build everything in the package, but not do it recursively.

@A-Sam
Copy link
Author

A-Sam commented Mar 19, 2019

Yeah. I use the query though to see the targets prior building and saving them to a file(smth like a dry-run). But I think it would be nice to add a feature on Bazel Build Targets to run all targets under a specific package in the extension.

allevato pushed a commit that referenced this issue Apr 9, 2019
- Add context command to the treeview on the packages to do build and testing
  of the package or the package resursively (:all vs /...).
- Includes a quickpick for generally using the command and selecting a package.

Mostly addresses #72, the limitation being the current UI doesn't really every
provide the "root" (the directory opened in VS Code) as a package, it just shows
the sub packages.
@thomasvl
Copy link
Member

thomasvl commented Apr 9, 2019

Now that we have the right click on the packages in the tree view, we could also change the treeview code to always show the opened directory node even when only one directory is open in VS Code, that catch is that directory might not have a BUILD file (it could be some random place within the overall bazel workspace). So building :all there would error, but ... would work, so the code would need to know to only enable some vs. all of the commands.

@A-Sam
Copy link
Author

A-Sam commented Apr 9, 2019

Bazel build //target/... will build all targets in the package which is what i originally meant and use.. thumbs up! Will this be released any time soon?

@thomasvl
Copy link
Member

thomasvl commented Jun 5, 2019

The 0.2.0 release in May should have had it.

@allevato allevato added the type: feature request New feature or request label Sep 24, 2019
@allevato
Copy link
Member

Closing because this was released in 0.2.0.

@oliverdain
Copy link

It does not seem to be possible to build //.... If I select Bazel build package recursively I have to select a package and // is not an option. I have tons of packages so it'd be really nice to be able to just build all of them.

@thomasvl
Copy link
Member

Please open a new issue specific to the whole WORKSPACE cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature request New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants