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

Feat release 1.5 #356

Closed
wants to merge 9 commits into from
Closed

Feat release 1.5 #356

wants to merge 9 commits into from

Conversation

juzhiyuan
Copy link
Member

@juzhiyuan juzhiyuan commented Aug 2, 2020

@apache/apisix-committers Please take a look, this PR mainly updates Deployment by manually, and added CHANGELOG for V1.5.

Once the PR is approved and merged, I will checkout a branch called v1.5

@juzhiyuan juzhiyuan requested review from gxthrj, asf-ci and moonming and removed request for asf-ci August 2, 2020 08:13
@rohityadavcloud
Copy link
Member

rohityadavcloud commented Aug 2, 2020

@juzhiyuan please avoid pinging all the apache committers (you've pinged 3k+ people)

@juzhiyuan
Copy link
Member Author

@juzhiyuan please avoid pinging all the apache committers (you've pinged 3k+ people)

Oh no.. I meant to ping only apisix's committers..

@ctubbsii
Copy link
Member

ctubbsii commented Aug 2, 2020

@juzhiyuan please avoid pinging all the apache committers (you've pinged 3k+ people)

Oh no.. I meant to ping only apisix's committers..

Hi @juzhiyuan . I'm not a committer on this project, but received the notification along with the other people throughout Apache, and wanted to offer a few friendly tips:

You never need to ping any GitHub teams on a PR. That forces a notification to everybody on that team who has linked their GitHub account to their Apache committer account, even if they've specifically elected to not receive notifications for activity on the repository. Those teams are primarily used internally for permissions management, and are not the best way to get the developers' attention.

Mentioning a team is also redundant. If somebody has chosen to watch the repository, then they would have received a notification of the new PR anyway, because they have subscribed to those notifications. No additional mention is necessary to trigger a notification.

In addition, the developer teams would have also received an email notification for new pull requests on their notification list, such as this one.

If you open a pull request and don't hear anything from the developers in a reasonable time, the best practice is probably to try to simply comment on the pull request or reach out to their developer mailing list instead (usually dev@<project>.apache.org), to request their attention.

People make mistakes. I'm sure it's not a huge problem, but I hope these tips help in future.

@juzhiyuan
Copy link
Member Author

Hi @ctubbsii , your tips would help me to use GitHub more reasonable, thanks a lot!

@apache apache locked as off-topic and limited conversation to collaborators Aug 2, 2020
@juzhiyuan
Copy link
Member Author

Because I pinged to the whole Apache committeers by mistake, I have to close this PR.

related issue #357

@juzhiyuan juzhiyuan closed this Aug 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants