bazelbuild / migration-tooling

Migration tools for Bazel
Apache License 2.0
45 stars 30 forks source link

Write output to generate_workspace.bzl by default, instead of WORKSPACE and BUILD files #4

Closed kchodorow closed 7 years ago

kchodorow commented 7 years ago

This splits the output writer into old & new versions. By default, generate_workspace will write generate_workspace.bzl, but users can specify --direct-to-ws to get the old behavior.

See BzlWriter.java#L54 for new usage info.

kchodorow commented 7 years ago

Wow, it is super annoying writing shell tests without unittest.bash. Working on that...

kchodorow commented 7 years ago

That's a no on the shell tests. It's too hard to get a copy of Bazel (sadly) and generate_workspace contact Maven Central, which seems suboptimal.

I added more info to the README and Michelle's currently got a CL in progress documenting this.

katre commented 7 years ago

Too bad on the shell tests, but the README is definitely helpful.