search

bazeltools / bazel-deps

Generate bazel dependencies for maven artifacts
MIT License
249 stars 121 forks source link

Include generateNeverlink in format-deps (#298) #300

Closed mcamenzind closed 3 years ago

mcamenzind commented 3 years ago

This fixes #298.

New behavior does not drop generateNeverlink during formatting.

johnynek commented 3 years ago

This is great.

Can we add a test that exercises this code (preferably one that would have failed before this test).

johnynek commented 3 years ago

I'll merge, but if you have time to add a test, I would really appreciate it.

(this is exercised, I think by a format command that the repo runs in CI, but I don't think there are any generateNeverlink in there.)