Skip to content

correct the some old info in contributing guide#633

Open
XinfengZhang wants to merge 1 commit intointel:masterfrom
XinfengZhang:contributing
Open

correct the some old info in contributing guide#633
XinfengZhang wants to merge 1 commit intointel:masterfrom
XinfengZhang:contributing

Conversation

@XinfengZhang
Copy link
Copy Markdown
Contributor

Signed-off-by: Carl Zhang carl.zhang@intel.com

Signed-off-by: Carl Zhang <carl.zhang@intel.com>
Comment thread CONTRIBUTING.md

Our github bug/enhancement backlog and work queue are tracked in a
[Libva waffle.io kanban](https://waffle.io/intel/libva).
If you have a problem, and is unsuitable to file issue, or is urgent to get support, you could send a mail to maintainers in https://github.com/orgs/intel/teams/libva-maintainers/members
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I am unsure that this link is visible by non intel and/or non-members.: if I am not signed in and click on it, github ask me to sign in explicitly. Can someone check whether it's actually accessible? @evelikov : can you, please, help?

Copy link
Copy Markdown
Contributor

@evelikov evelikov Sep 27, 2022

Choose a reason for hiding this comment

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

That's correct - people outside of the (Intel?) team cannot open the URL.

Here's my understanding and links to another project for illustration:
Github shows people part of a namespace/org (assuming those people didn't set themselves as hidden) - for example see the "People" section in https://github.com/mpv-player/ and the complete list in https://github.com/orgs/mpv-player/people. AFAICT Github do not allow listing of team members for a given project within the namespace - there is no People section in https://github.com/mpv-player/mpv

Aside: this MR is out of date - the ML and waffle references got removed a week or two ago. Sorry didn't see the information was duplicated across the README and CONTRIBUTING files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants