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

Consider exposing an aspect CLI plugin #34

Open
alexeagle opened this issue Dec 6, 2022 · 1 comment
Open

Consider exposing an aspect CLI plugin #34

alexeagle opened this issue Dec 6, 2022 · 1 comment

Comments

@alexeagle
Copy link
Collaborator

We could add a third entry point next to target-determinator and driver which would contribute a command to Bazel, by implementing a https://docs.aspect.build/aspect-build/aspect-cli/5.0.1/docs/help/topics/plugins.html

If you're up for it, I could get this started. I'd like to be able to run target-determinator just with bazel affected.

@illicitonion
Copy link
Collaborator

Very happy for this, particularly to review PRs! IIRC @shs96c may have some prior art to build on here too.

One thing to call out is we explicitly added a BazelCmd interface to make something like an Aspect plugin easier to write, but note that we need the ability to run bazel from a working directory other than the workspace, which IIRC was an API difference.

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

No branches or pull requests

2 participants