Skip to content

First draft for the Introductory Contributor Section Article #67 - #98

Merged
rrrutledge merged 9 commits into
masterfrom
contributor/introduction
Sep 12, 2019
Merged

rrrutledge merged 9 commits into
masterfrom
contributor/introduction

Conversation

@lenucksi

@lenucksi lenucksi commented Jul 24, 2019 •

Copy link
Copy Markdown
Member

Content

This is an initial draft for the "The Contributor in InnerSource" article based upon the outline and video proposal document.
Feedback, Change proposals either using the GitHub change feature or as pushes to the branch for larger changes (hence the direct push to this repo) are more than welcome.

Technical detail

  • I've included headers from the outline document and video proposal for drafting purposes. Once they document is ready, those should obviously disappear.
  • The referenced articles currently have no URL yet, thus the link fields are currently left blank. They need to be replaced once they are ready. An initial approach could be to link the GitHub articles.

Comment thread contributor/01-introduction-article.md Outdated
- We argue that the Committer is part of the guest team and needs to fit in with the host team.
- In a more-than-one-person project what do we call the people aside of the TC/PO that are members of the host team? Just team members?
- OSS would likely not make this distinction and everyone would end up as contributor until it's committer / trusted committer at some point I guess?
- In the ASF-Rule-set this would be the distinction between Developer and Committer.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@rrrutledge

Copy link
Copy Markdown
Contributor

Thanks for this pull request, @lenucksi. I think that I'll be able to have a look before this week's call for the learning path.

@lenucksi
lenucksi requested a review from rrrutledge August 6, 2019 18:32
@rrrutledge

rrrutledge commented Aug 8, 2019 •

Copy link
Copy Markdown
Contributor

An initial approach could be to link the GitHub articles.

Yes we should do this

@rrrutledge rrrutledge left a comment

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.

Thanks so much for doing this article @lenucksi. I like the way that you speak with a strong, friendly, guiding voice to the reader.

Left a few comments. Also, suggest shorter headings.

Comment thread contributor/01-introduction-article.md Outdated
[EDITOR NOTE:
- In other articles we draw a distinction between guest and host team.
- We argue that the Committer is part of the guest team and needs to fit in with the host team.
- In a more-than-one-person project what do we call the people aside of the TC/PO that are members of the host team? Just team 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 think that something like host team members is fine. Is this concept used somewhere in this article?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We're using that concept implicitly in lines 41,42 and explicitly in line 55.
Also, we're announcing to use the concept extensively in line 85.

And since home/guest might establish a mental concept of separation (us/them) I think that we should give that a aspect a consideration to see if and how we want to deal with it throughout the content.
This could enhance clarity and prevent contradictory information later on. (Me being a nitpick and definition pedant here ;))

I've suggested the Apache source below since that gives an example for this

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 think in a corporate context we assume that division is already built in by the organization. We want to bring it together and suggest the terminology of Host/Guest to encourage people to give warm welcome and collaboration to each other.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sounds good to me.
So we're using host team member for a person that is part of the host team but no trusted committer (yet) then?

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.

Yes, I think so.

Comment thread contributor/01-introduction-article.md Outdated
Comment thread contributor/01-introduction-article.md Outdated

@lenucksi lenucksi left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Addressed your review comments.

Comment thread contributor/01-introduction-article.md Outdated
Comment thread contributor/01-introduction-article.md Outdated
Comment thread contributor/01-introduction-article.md Outdated
[EDITOR NOTE:
- In other articles we draw a distinction between guest and host team.
- We argue that the Committer is part of the guest team and needs to fit in with the host team.
- In a more-than-one-person project what do we call the people aside of the TC/PO that are members of the host team? Just team members?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We're using that concept implicitly in lines 41,42 and explicitly in line 55.
Also, we're announcing to use the concept extensively in line 85.

And since home/guest might establish a mental concept of separation (us/them) I think that we should give that a aspect a consideration to see if and how we want to deal with it throughout the content.
This could enhance clarity and prevent contradictory information later on. (Me being a nitpick and definition pedant here ;))

I've suggested the Apache source below since that gives an example for this

Comment thread contributor/01-introduction-article.md Outdated

@lenucksi lenucksi left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Left a few comments. Also, suggest shorter headings.

The titles are actually taken verbatim from the outline. I'm happy for suggestions to shorten them - and maybe I'll come up with something shorter myself.

@rrrutledge rrrutledge mentioned this pull request Aug 12, 2019
@rrrutledge

Copy link
Copy Markdown
Contributor

I'm happy for suggestions

Left in #110.

To appeal to an individual who just cares about making progress on their project.
Avoid explicitly bringing up contributions outside of repo contents.

@rrrutledge rrrutledge left a comment

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.

Responded to all comments.

Comment thread contributor/01-introduction-article.md Outdated
Comment thread contributor/01-introduction-article.md Outdated
[EDITOR NOTE:
- In other articles we draw a distinction between guest and host team.
- We argue that the Committer is part of the guest team and needs to fit in with the host team.
- In a more-than-one-person project what do we call the people aside of the TC/PO that are members of the host team? Just team 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 think in a corporate context we assume that division is already built in by the organization. We want to bring it together and suggest the terminology of Host/Guest to encourage people to give warm welcome and collaboration to each other.

Comment thread contributor/01-introduction-article.md Outdated

@rrrutledge rrrutledge left a comment

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.

Almost done!

Comment thread contributor/01-introduction-article.md Outdated
[EDITOR NOTE:
- In other articles we draw a distinction between guest and host team.
- We argue that the Committer is part of the guest team and needs to fit in with the host team.
- In a more-than-one-person project what do we call the people aside of the TC/PO that are members of the host team? Just team 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.

Yes, I think so.

Comment thread contributor/01-introduction-article.md Outdated
…ion-persona

Rewrite introductory paragraphs

@rrrutledge rrrutledge left a comment

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.

Looks good to me. We can revisit the EDITOR NOTE later.

@lenucksi

Copy link
Copy Markdown
Member Author

Looks good to me. We can revisit the EDITOR NOTE later.

Thanks, I'd say we put this up for merge discussion on the next call.
I'd think the EDITOR NOTE was dealt with the term host team member according to this.
What do the others think?

(And we can still add new PRs on top of this later.)

@rrrutledge

Copy link
Copy Markdown
Contributor

I'd say we put this up for merge discussion on the next call.

🆗

@rrrutledge

Copy link
Copy Markdown
Contributor

@lenucksi we forgot on the call to discuss this article, specifically, if it can be merged. If you think so then go ahead.

@rrrutledge
rrrutledge merged commit 3281d93 into master Sep 12, 2019
@rrrutledge
rrrutledge deleted the contributor/introduction branch September 12, 2019 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants