Jetz72
Jetz72
Ah, that'd explain why this hasn't come up more. Could have sworn I turned that feature off but I guess not. Tangential note, it might be worthwhile to add an...
Finally got the nasty part of this migration done, primarily [ #76aa196 ](https://github.com/Card-Forge/forge/pull/6047/commits/76aa19622ec65db5a1e71fed1792c183a97818a5). Some cases were clear improvements when migrated from Iterables to Streams. Some cases were skipped because they...
FCollectionView could use some fleshing out. Not sure whether that means adding a stream method, implementing a more generally useful interface in it, or dropping it entirely. But I didn't...
Added `StreamUtils.random` and converted a handful of uses of `Aggregates.random` to it instead. I didn't bother changing the ones that just passed the source collection right into the method, just...
I looked at cutting out more of Guava and recreating the functionality within other util classes, but the scope of that started expanding too quickly for this PR at least....
Gah. I was excited when I saw we'd moved up to Java 17. Should I just treat the language level as though it were 11, then?
Android 12 and 13 reportedly support up to Java 11, and Android 14 partially supports Java 17. https://developer.android.com/build/jdks#compileSdk From what I can tell, the D8 compiler has support for backporting...
I've encountered the issue again under different circumstances. This issue no longer seems specific to morph. Cancelling any payment after a spell is copied through a play effect seems to...
Glad to see those properly documented. Might function fine to just change the set code to the one Scryfall uses? Would be good to cache them on the image server...
Perhaps you could leave it to the user to specify their preferred common parent for the variable when they name it in the `catch` clause, if there's more than one...