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
I like the plugin but I ran into a couple things that would be nice to have (especially during first-time use):
Small but took a minute to find - I am used to camelCase with Java and Groovy names, so I started out with testLogger eventually realized the L is not capitalized. Maybe make both valid or just point out the difference in the documentation.
My projects use parallel tests which caused the theme to fail. It appears to know that I am using parallel testing - if this is the case, could you not switch to the parallel version of the same theme and provide a warning log that this change was made.
These are just suggestions - feel free to close this issue if they are not something you want to pursue.
The text was updated successfully, but these errors were encountered:
Thanks for the feedback. Regarding the first point, I chose all lowercase inspired by Gradle's own convention for some DSLs (subprojects, buildscript etc). But in retrospect testlogger does look a bit dated. I will think about supporting both testlogger and testLogger.
Your second point about switching to a corresponding parallel theme is interesting. Let me consider that too.
I would like the plugin's behaviour to be predictable. Therefore, I am going to abandon the idea of automatically switching themes based on Gradle settings, at least for now.
I like the plugin but I ran into a couple things that would be nice to have (especially during first-time use):
testLogger
eventually realized theL
is not capitalized. Maybe make both valid or just point out the difference in the documentation.These are just suggestions - feel free to close this issue if they are not something you want to pursue.
The text was updated successfully, but these errors were encountered: