A few things could be improved in our samples, eg:
mainClassName Gradle property is deprecated, could use mainClass.set(...)
every sample should set options.addInAppInclude("io.sentry.samples"); if not set automatically (Android does), so we get proper inApp frames.
this could be done either via options as described above or sentry.properties file.
samples-jul is not reporting events at all.
call Sentry.flush if sample has a short lifecycle, which might be the case for Console, JUL etc samples, maybe the ShutdownHookIntegration does it automatically, but I'm not sure that this hook is invoked for every type of App, it does not for Android for instance.
cc @maciejwalkowiak
A few things could be improved in our samples, eg:
mainClassNameGradle property is deprecated, could usemainClass.set(...)every sample should set
options.addInAppInclude("io.sentry.samples");if not set automatically (Android does), so we get proper inApp frames.this could be done either via options as described above or
sentry.propertiesfile.samples-julis not reporting events at all.call
Sentry.flushif sample has a short lifecycle, which might be the case for Console, JUL etc samples, maybe theShutdownHookIntegrationdoes it automatically, but I'm not sure that this hook is invoked for every type of App, it does not for Android for instance.cc @maciejwalkowiak