Skip to content
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

Add additional configuration for Sentry #938

Merged
merged 4 commits into from
Sep 14, 2023
Merged

Add additional configuration for Sentry #938

merged 4 commits into from
Sep 14, 2023

Conversation

pierredup
Copy link
Member

No description provided.

@pierredup pierredup added this to the 2.2.6 milestone Sep 14, 2023
@pierredup pierredup self-assigned this Sep 14, 2023
@what-the-diff
Copy link

what-the-diff bot commented Sep 14, 2023

PR Summary

  • Added new environment variables for Sentry
    New environment variables have been included to configure the Sentry, which is a tool that helps monitor and fix crashes in real time.

  • Updates to sentry and services configuration
    The configuration files of Sentry and of the service are updated to include these changes. This will control how and when the application will send out data for real time crash analysis.

  • Added new file for Cron Schedule Events
    A new PHP file named CronScheduleEventSubscriber.php has been added. This file is responsible for subscribing to, and responding to, scheduled tasks (or Cron) events that happen within our application.

  • Registered the new service in the services configuration
    The newly added CronScheduleEventSubscriber has been registered as a service in the services.yml configuration file. This means that it will now be globally available in the application, and can be utilized wherever required.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@codecov
Copy link

codecov bot commented Sep 14, 2023

Codecov Report

Patch coverage: 21.05% and project coverage change: -0.20% ⚠️

Comparison is base (5d115d0) 50.97% compared to head (2a0f1e5) 50.77%.
Report is 11 commits behind head on 2.2.x.

Additional details and impacted files
@@             Coverage Diff              @@
##              2.2.x     #938      +/-   ##
============================================
- Coverage     50.97%   50.77%   -0.20%     
- Complexity     2229     2239      +10     
============================================
  Files           442      443       +1     
  Lines          8295     8337      +42     
============================================
+ Hits           4228     4233       +5     
- Misses         4067     4104      +37     
Flag Coverage Δ
unittests 50.77% <21.05%> (-0.20%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
src/CoreBundle/ConfigWriter.php 0.00% <0.00%> (ø)
src/CoreBundle/Repository/VersionRepository.php 36.84% <0.00%> (ø)
...le/EventSubscriber/CronScheduleEventSubscriber.php 0.00% <0.00%> (ø)
src/InstallBundle/Action/Install.php 0.00% <0.00%> (ø)
src/InstallBundle/Command/InstallCommand.php 0.00% <0.00%> (ø)
src/UserBundle/Email/ResetPasswordEmail.php 0.00% <0.00%> (ø)
src/UserBundle/Repository/UserRepository.php 95.12% <0.00%> (ø)
src/CoreBundle/Config/Loader/EnvLoader.php 71.42% <63.63%> (-14.29%) ⬇️
src/InstallBundle/Listener/RequestListener.php 96.66% <100.00%> (ø)
src/InstallBundle/Listener/UpgradeListener.php 56.25% <100.00%> (ø)
... and 2 more

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pierredup pierredup merged commit 6f0654e into 2.2.x Sep 14, 2023
15 of 20 checks passed
@pierredup pierredup deleted the sentry-config branch September 14, 2023 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant