Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove slacknotifer #492

Open
wants to merge 2 commits into
base: staging
Choose a base branch
from
Open

Remove slacknotifer #492

wants to merge 2 commits into from

Conversation

lodewiges
Copy link
Contributor

@lodewiges lodewiges commented Feb 7, 2025

This PR removes the slacknotifer gem due to it not being used anymore. fixes #490

It also enables the daily verse to work in the development environment. https://github.com/csvalpha/development-environment/issues/12

Copy link

codecov bot commented Feb 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.92%. Comparing base (b02963f) to head (524323e).

Additional details and impacted files
@@             Coverage Diff             @@
##           staging     #492      +/-   ##
===========================================
- Coverage    99.92%   99.92%   -0.01%     
===========================================
  Files          203      201       -2     
  Lines         2772     2761      -11     
===========================================
- Hits          2770     2759      -11     
  Misses           2        2              

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

@lodewiges lodewiges self-assigned this Feb 18, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove slack notifier
1 participant