Fabian Meumertzheim
Fabian Meumertzheim
I kind of like this as it reduces the amount of Gazelle-specific configuration. I don't know which library is best, but we should definitely use one rather than roll our...
Based on https://github.com/aspect-build/aspect-cli-legacy/issues/52, what do you expect to be the use case for this if we can't enable it by default due to performance issues? It's surprising to me that...
I appreciate the effort you put into this, but yes, I think that we should support `ignore_directories` instead. As you remarked, it's simpler but still expressive enough to be generally...
It should work with a regex like this: https://github.com/bazel-contrib/bazel-gazelle/blob/2a84bad644e0657346ad75704c2a4835080a3cdf/resolve/resolve_test.go#L30 We could automatically wrap the regex in `^...$` if it isn't already.
Cc @lkassar-stripe
I agree that the PR you linked has unintended consequences for existing usage patterns and we should fix that. What do you think of applying `ReplaceAllString` only if the regex...
I'm a bit swamped right now, would you be willing to send a PR to implement some variant of this approach?
I'll bring this up at the next Rules Authors SIG meeting.
We discussed this in the Rules Authors SIG meeting and would consider funding the development of a standalone tool (e.g. in Go or Rust) that consumes a segmented module file...
Thanks for the detailed report. Looks like we "just" have to fix 3. - 7. one by one? Do you already have a lead on some of these issues?