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

Collaboration and Contribution Process Review #864

Closed
bnb opened this issue May 11, 2020 · 18 comments
Closed

Collaboration and Contribution Process Review #864

bnb opened this issue May 11, 2020 · 18 comments

Comments

@bnb
Copy link
Contributor

bnb commented May 11, 2020

I briefly reached out to @mcollina about this who suggested I open an issue on the TSC repo:

I'm not sure if we've ever done a full review/reflective pass on our collaboration and contribution documents within the project (for example, the TSC Charter and Working Groups doc,).

It may perhaps be beneficial to collectively take a look at these processes (and any others!) and see both which parts may or may not be reflective of how we currently operate and which parts could be improved to allow us both as a project to work better and for the community to contribute effectively and meaningfully.

@Trott
Copy link
Member

Trott commented May 11, 2020

I think we're overdue to move whatever is of value in Applications elsewhere and then remove the directory in this repository. We don't have projects applying for membership anymore. They should go to the OpenJS Foundation instead, I would think.

@jasnell
Copy link
Member

jasnell commented May 11, 2020

It's going to be easiest to start with a list of issues that may need to be addressed, otherwise it's going to be way too open ended.

@mcollina
Copy link
Member

@bnb would you like to faciltiate this? I think having an approach from outside the folks with Node.js commit access would be very interesting.

@boneskull
Copy link

(also collaborators who would contribute more often, if not for the process. 🙂)

@bnb
Copy link
Contributor Author

bnb commented May 14, 2020

@mcollina sure. Is this something you'd want me to do inside of the TSC repo, inside of the Admin repo, or somewhere else?

(I am also OOO this week so may be slow to respond, just wanted to check on this to see if I was blocking at all)

@mcollina
Copy link
Member

I think opening issues here is ok.

Having a fresh perspective on https://github.com/nodejs/node/blob/master/CONTRIBUTING.md and how we could simplify things.

@Trott
Copy link
Member

Trott commented May 14, 2020

Having a fresh perspective on https://github.com/nodejs/node/blob/master/CONTRIBUTING.md and how we could simplify things.

Since most of us who are frequent contributors to the project also never look at that file, it's possible there's some pretty low-hanging fruit in there. My immediate suggestions:

  • Simplify language everywhere. Be brief, informative, and friendly.
  • For issues:
    • Remove "How to Contribute in Issues".
    • Remove "Resolving a Bug Report".
  • For pull requests:
    • Remove the preamble that is before the table of contents.
    • Remove the lengthy section on "Reviewing pull requests". It's not that it isn't important; it's just irrelevant to someone wanting to open a pull request, which is who is most likely to be reading this. If we want to keep it, link to it in the collaborators guide rather than having it all laid out here.

jasnell added a commit that referenced this issue May 14, 2020
Building on the discussion that has been started in #864, I'd like to propose that we take a queue from other significant open source projects and conduct our own Contributor Survey -- with the intent of helping us to determine where we can continue to make improvements to our contribution process.

Using the 2019 Kubernetes Contributor Survey as an initial starting point, I've put together an initial set of questions that can allow us to start structuring something.

This is a draft. The intent here is to build / refine the list of questions we'd like to see on a survey should we decide to do one.

Please do not yet merge this PR. It is still a work in progress
jasnell added a commit that referenced this issue May 27, 2020
Building on the discussion that has been started in
#864, I'd like to
propose that we take a queue from other significant open
source projects and conduct our own Contributor Survey
-- with the intent of helping us to determine where we
can continue to make improvements to our contribution process.

Using the 2019 Kubernetes Contributor Survey as an initial
starting point, I've put together an initial set of questions
that can allow us to start structuring something.
@mhdawson
Copy link
Member

mhdawson commented Oct 8, 2020

@bnb are you still planning to move this forward ?

@bnb
Copy link
Contributor Author

bnb commented Oct 8, 2020

@mhdawson good timing on this question 😅

I'd completely not remembers that I created it nor that I committed to it. After next week I'm planning on pretty aggressively focusing on open-source work for a bit, and I can prioritize this in that time.

@mmarchini
Copy link
Contributor

That's something I've been thinking a lot about lately (especially in the context of https://github.com/nodejs/node/blob/master/CONTRIBUTING.md), and I'm eager to bounce off ideas with anyone who's also interested in this conversation :D

@mhdawson
Copy link
Member

mhdawson commented Oct 8, 2020

@bnb @mmarchini great to hear :)

@mhdawson
Copy link
Member

mhdawson commented Jan 4, 2021

@bnb, @mmarchini anything to report on this front?

@mmarchini
Copy link
Contributor

Haven't done anything on this front lately.

Trott added a commit to Trott/io.js that referenced this issue Jan 14, 2021
Remove "How to Contribute in Issues". This is not Node.js-specific and
is likely to cause many readers to tune out. If we want to include this
kind of how-all-issue-trackers-are-intended-to-work information, let's
link to an external source. But I think it's OK to simply remove it.

Refs: nodejs/TSC#864 (comment)

PR-URL: nodejs#36891
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Trott added a commit to Trott/io.js that referenced this issue Jan 16, 2021
This section isn't particularly useful in this context and contributes
to making the document longer and less effective. This is part of a
larger effort to make the contributing docs brief, informative, and
friendly.

Refs: nodejs/TSC#864 (comment)

PR-URL: nodejs#36905
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Trott added a commit to Trott/io.js that referenced this issue Jan 16, 2021
jasnell pushed a commit to nodejs/node that referenced this issue Jan 18, 2021
Refs: nodejs/TSC#864 (comment)

PR-URL: #36960
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
ruyadorno pushed a commit to nodejs/node that referenced this issue Jan 22, 2021
Remove "How to Contribute in Issues". This is not Node.js-specific and
is likely to cause many readers to tune out. If we want to include this
kind of how-all-issue-trackers-are-intended-to-work information, let's
link to an external source. But I think it's OK to simply remove it.

Refs: nodejs/TSC#864 (comment)

PR-URL: #36891
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
ruyadorno pushed a commit to nodejs/node that referenced this issue Jan 22, 2021
This section isn't particularly useful in this context and contributes
to making the document longer and less effective. This is part of a
larger effort to make the contributing docs brief, informative, and
friendly.

Refs: nodejs/TSC#864 (comment)

PR-URL: #36905
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
ruyadorno pushed a commit to nodejs/node that referenced this issue Jan 22, 2021
Refs: nodejs/TSC#864 (comment)

PR-URL: #36960
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@bnb
Copy link
Contributor Author

bnb commented Jan 22, 2021

I've not gotten a chance to work on this. Perhaps taking a slightly different approach - would anyone be interested in sitting down and pairing on it? If not, that's fine ❤️

@mhdawson
Copy link
Member

I wonder if it might fit into - nodejs/next-10#16. We could target one of the upcoming Next-10 meetings to discuss? That would be a slightly larger pairing :)

ruyadorno pushed a commit to nodejs/node that referenced this issue Jan 25, 2021
Refs: nodejs/TSC#864 (comment)

PR-URL: #36960
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@alikkille

This comment has been minimized.

targos pushed a commit to nodejs/node that referenced this issue May 1, 2021
Remove "How to Contribute in Issues". This is not Node.js-specific and
is likely to cause many readers to tune out. If we want to include this
kind of how-all-issue-trackers-are-intended-to-work information, let's
link to an external source. But I think it's OK to simply remove it.

Refs: nodejs/TSC#864 (comment)

PR-URL: #36891
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
targos pushed a commit to nodejs/node that referenced this issue May 1, 2021
This section isn't particularly useful in this context and contributes
to making the document longer and less effective. This is part of a
larger effort to make the contributing docs brief, informative, and
friendly.

Refs: nodejs/TSC#864 (comment)

PR-URL: #36905
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
targos pushed a commit to nodejs/node that referenced this issue May 1, 2021
Refs: nodejs/TSC#864 (comment)

PR-URL: #36960
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@mhdawson
Copy link
Member

The last comment on this was more than a year ago, while it would be great to make progress that's not happened for quite some time. @bnb any objections to be closing this issue?

@bnb
Copy link
Contributor Author

bnb commented Jun 28, 2022

I still think we should do this but happy to close.

@bnb bnb closed this as completed Jun 28, 2022
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

No branches or pull requests

8 participants