-
Notifications
You must be signed in to change notification settings - Fork 10
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
Enable toggling for codecov slack app notifications #88
Conversation
c24beab
to
71767d4
Compare
Codecov Report
Changes have been made to critical files, which contain lines commonly executed in production. Learn more @@ Coverage Diff @@
## main #88 +/- ##
=======================================
Coverage 98.48% 98.48%
=======================================
Files 364 364
Lines 26818 26826 +8
=======================================
+ Hits 26412 26420 +8
Misses 406 406
Flags with carried forward coverage won't be shown. Click here to find out more.
|
71767d4
to
7556cd8
Compare
7556cd8
to
ff08642
Compare
decoration_type=self.decoration_type, | ||
) | ||
# yield notifier if slack_app field is True, nonexistent, or a non-empty dict | ||
slack_app_yaml_field = read_yaml_field(self.current_yaml, ("slack_app",), True) |
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.
does this makes the slack app enabled by default?
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.
yes
ff08642
to
8e9e6d1
Compare
This commit enables toggling for the slack app notifications via the codecov config slack_app option. Signed-off-by: joseph-sentry <[email protected]>
8e9e6d1
to
9968b87
Compare
This PR enables toggling for the slack app notifications via the codecov config slack_app option.
Requires updating shared to use the new codecov schema containing the slack_app option.
Fixes: codecov/engineering-team#401
Depends on: codecov/shared#34