-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sentry integration #116
Sentry integration #116
Conversation
WalkthroughThe updates primarily focus on integrating Sentry for enhanced error tracking and debugging capabilities within the application. This includes adding the Sentry plugin, configuring it to enrich error reporting with source context, and ensuring source code is uploaded for better insight. Additionally, minor updates were made to the testing dependencies, alongside a crucial import statement for Sentry in the main application class to kickstart its functionality. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 1
Configuration used: CodeRabbit UI
Files ignored due to path filters (1)
src/main/resources/application.yml
is excluded by:!**/*.yml
Files selected for processing (2)
- build.gradle (3 hunks)
- src/main/java/io/kontur/disasterninja/DisasterNinjaApplication.java (1 hunks)
Additional comments: 3
src/main/java/io/kontur/disasterninja/DisasterNinjaApplication.java (1)
- 3-3: The addition of the
io.sentry.Sentry
import statement aligns with the PR's objective of integrating Sentry for error tracking and performance monitoring. Ensure that Sentry is properly initialized and used where necessary in the application to fully leverage its capabilities.build.gradle (2)
- 8-8: The addition of the Sentry JVM Gradle plugin (version 4.3.0) is correctly implemented. This plugin is essential for enabling Sentry's error tracking and performance monitoring capabilities within the JVM environment of the project.
- 74-84: The tasks configured for Sentry source code integration are set up to ensure they run in the correct order relative to the Apollo source generation tasks. This setup is crucial for ensuring that Sentry's source code integration does not interfere with or is negatively impacted by other build processes. Good job ensuring that these dependencies are correctly managed.
38355fa
to
9b87f89
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files ignored due to path filters (1)
src/main/resources/application.yml
is excluded by:!**/*.yml
Files selected for processing (2)
- build.gradle (3 hunks)
- src/main/java/io/kontur/disasterninja/DisasterNinjaApplication.java (1 hunks)
Files skipped from review as they are similar to previous changes (2)
- build.gradle
- src/main/java/io/kontur/disasterninja/DisasterNinjaApplication.java
Summary by CodeRabbit