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

update contact information on the repo and release package #3383

Merged
merged 4 commits into from
Aug 20, 2024
Merged

Conversation

ekzhu
Copy link
Collaborator

@ekzhu ekzhu commented Aug 19, 2024

(1) Replacing an individual outlook.com email address with a mailing list containing all current repo maintainers
(2) Remove text and keep Microsoft open-source statement

@gagb gagb requested review from afourney and removed request for afourney August 19, 2024 23:28
setup.py Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
Copy link
Collaborator

@randombet randombet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change need more context and supporting evidence to justify itself.
It contains changes that may jeopardize the contributor community.

@microsoft microsoft locked and limited conversation to collaborators Aug 20, 2024
@ekzhu ekzhu enabled auto-merge August 20, 2024 01:46
@ekzhu ekzhu dismissed stale reviews from yiranwu0 and qingyun-wu August 20, 2024 02:04

See my reply.

@ekzhu ekzhu added this pull request to the merge queue Aug 20, 2024
Merged via the queue into main with commit d5068d9 Aug 20, 2024
134 of 142 checks passed
@ekzhu ekzhu deleted the update-contact branch August 20, 2024 02:10
@gagb
Copy link
Collaborator

gagb commented Aug 20, 2024

This change need more context and supporting evidence to justify itself. It contains changes that may jeopardize the contributor community.

@randombet -- a PR for acknowledging contributors in the repo is already on the way. Stay tuned. The PR only removed non-standard wording and created a distribution list for maintainers instead of a private outlook.com address.

Edit: #3384

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.

8 participants