-
Notifications
You must be signed in to change notification settings - Fork 7
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
Feedback dependency management approach from here into bazel core. #10
Comments
Hi @hsyed thanks for the feedback. Sounds like a pain point for you is copying back the transitive closure into the rule, which you are proposing some sort of lock file. I agree it's a little annoying, lock file could be a good idea. However, let's say for a moment that the lockfile had the format So, question is, how to make lock file maintenance less annoying than the current state of affairs? |
@hsyed ping |
@pcj sorry for the delay. If we had a lock file of some form couldn't we generate it and get rid of the copy pasting ? The user manages tuning the dependencies but beyond that the boilerplate could be generated. Taking some inspiration from the go rules gazelle tool. Could we express dependencies in a yaml file with the high level co-ords of along with force entries. This repo could provide a tool of some form that generates two skylark files.
Would this be doable ? |
Have you tried https://github.com/johnynek/bazel-deps by @johnynek? That approach is fairly similar to what you are describing. |
I'm hoping we can work some parts of the dependency management approach here into bazel core.
Bridge to issue in bazel core / migration-tooling.
The text was updated successfully, but these errors were encountered: