You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PS: one small suggestion though, you may have noticed that I did include not just core, but also pattern explicitly, as having it pulled transitively by gradle does not make it accessible as implementation even if io.jstach.rainbowgum:rainbowgum is included as such. I think to have a better developer ergonomic, when including io.jstach.rainbowgum:rainbowgum, the transitive dependencies should be pulled in with the same scope so that having implementation("io.jstach.rainbowgum:rainbowgum:0.2.0") is enough. When just wanting to use the defaults, having runtimeOnly("io.jstach.rainbowgum:rainbowgum:0.2.0") should be sufficient.
PS: one small suggestion though, you may have noticed that I did include not just
core
, but alsopattern
explicitly, as having it pulled transitively bygradle
does not make it accessible asimplementation
even ifio.jstach.rainbowgum:rainbowgum
is included as such. I think to have a better developer ergonomic, when includingio.jstach.rainbowgum:rainbowgum
, the transitive dependencies should be pulled in with the same scope so that havingimplementation("io.jstach.rainbowgum:rainbowgum:0.2.0")
is enough. When just wanting to use the defaults, havingruntimeOnly("io.jstach.rainbowgum:rainbowgum:0.2.0")
should be sufficient.Originally posted by @cmdjulian in #55 (comment)
The text was updated successfully, but these errors were encountered: