-
Notifications
You must be signed in to change notification settings - Fork 484
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 mailing list to README #1023
Conversation
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. One comment inline.
Co-authored-by: tison <wander4096@gmail.com>
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
@PragmaTwice we can have a path filter for skipping CI on docs change. But since we required the status, it can be a bit tricky and we should make use of https://github.com/dorny/paths-filter to skip steps. It's not a requirement of this patch, of course. |
Sure, we need to skip CI if no actual code changes. |
Let me create an issue for it. |
* Add mailing list to README * Update .asf.yaml * Update README.md Co-authored-by: tison <wander4096@gmail.com> Co-authored-by: tison <wander4096@gmail.com>
Since
dev@
is already overwhelmed with discussion messages, and I always get duplicated discussion notification mails from both github anddev@
), I thought I could move the discussion notifications toissues@
as well.I also removed the google group link because it has not been used for a long time and many developers canot get notifications. Is this appropriate? @git-hulk