Billy Autrey

Results 20 comments of Billy Autrey

This config makes sense to me, but it doesn't seem immediately clear why this is behaving this way. Do you happen to have a minimal reproducer I can check out?

Hey @renatocaval, Sure! I can give it a shot.

> I squashed my commit into one and `Check / Code Style Docs / { "java": "8" }` is throwing that `AllowedIPFilter.md` hasn't a valid header but it has a...

> An empty comment so that this will not be part of my "review requests" list. > > Sorry for the noise, @mkurz. Hey Marcos, been a while! Hope all...

My team actually hit this exact issue as well. Adding that flag allowed us to proceed with further testing on Bazel 5.

> That said I'm not convinced this is evidence a bug per se; as I said, the root of the issue is that I discovered tests depending on other tests...

This makes me think that there's a deeper bug in the way that the aspect is looking for scala source files.

Yeah, I described exactly what I did above. Here's the line as it exists today: https://github.com/bazelbuild/intellij/blob/68bc3a7e12181ecc36d557db9379111841aa5192/aspect/testing/tests/src/com/google/idea/blaze/aspect/scala/scalalibrary/ScalaLibraryTest.java#L43 And here's the Java version of the same assertion: https://github.com/bazelbuild/intellij/blob/68bc3a7e12181ecc36d557db9379111841aa5192/aspect/testing/tests/src/com/google/idea/blaze/aspect/java/javalibrary/JavaLibraryTest.java#L91 If you change the...

@liucijus yeah that's where we were headed. We actually got Scala sources to attach with some very minor changes, I couldn't get your changes to work in the default bazel...

@raboof greetings! Is there anything on the roadmap to provide a `HttpsConnectionContext` for an HTTP/2 client? Seems like it's still missing.