-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add a Rails app to the examples folder if and when possible. #144
Comments
I think it would be great to have a Rails example, PRs are welcome! |
I was just thinking through what it would mean to put a Rails app in bazel and landed here : D Agreed, Id love to see the intended way to split code to be more 'bazely' for a rails app. |
@J-Swift I don't think there is that much to be done to make Rails work via Bazel. In fact, there should not be a big need to change the codebase structure in any way except for adding |
I was thinking more about trying to actually leverage bazel in a rails app so you could eg slice up the test suite and not run the whole thing |
That's what I mean by "making the dependencies more explicit". For example, you can create separate targets for |
I guess I just think rails apps typically are not operating with such explicit boundaries and so really the only way I imagined it being possible is if things were split into code modules (eg engines, or gems) |
Nothing really stops you from doing the dependencies only with Bazel. In this case, during the test execution, Bazel will symlink dependencies into a test |
Ok interesting Ill have to test it out when I get time |
Not sure if the rules can already be used on a Rails App, but I think that this should be ultimately a goal because so many huge monolithic applications can benefit from partial Bazel builds (that would likely be mostly running tests on sub-sections of the app that changed).
Perhaps leave this as a placeholder for now, or comment why this isn't a goal, or what do you think about Rails support down the road.
The text was updated successfully, but these errors were encountered: