-
Notifications
You must be signed in to change notification settings - Fork 527
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
fix:asf invalid notification scheme 'discussions_status' #2247
fix:asf invalid notification scheme 'discussions_status' #2247
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2247 +/- ##
=============================================
- Coverage 65.03% 53.61% -11.43%
+ Complexity 979 378 -601
=============================================
Files 498 498
Lines 40682 40684 +2
Branches 5681 5681
=============================================
- Hits 26458 21812 -4646
- Misses 11596 16567 +4971
+ Partials 2628 2305 -323 see 133 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
LGTM
@@ -57,5 +57,5 @@ notifications: | |||
issues: [email protected] | |||
issues_status: [email protected] | |||
issues_comment: [email protected] | |||
discussions_status: [email protected] | |||
discussions: [email protected] | |||
discussions_comment: [email protected] |
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.
@Z-HUANT we need also remove discussions_comment
for same problem
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.
Purpose of the PR
Error while running notifications feature from .asf.yaml in incubator-hugegraph!

Main Changes
fix:asf invalid notification scheme 'discussions_status'
Verifying these changes
Does this PR potentially affect the following parts?
fix:asf invalid notification scheme 'discussions_status'
Documentation Status
Doc - TODO
Doc - Done
Doc - No Need