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

Draft Notes for the OSS Core Meeting #52

Merged
merged 5 commits into from
Nov 22, 2018
Merged

Conversation

kelset
Copy link
Member

@kelset kelset commented Nov 11, 2018

Please review and LMK 😊

@kelset kelset added the 📑 Meeting This indicates an issue/PR related to a core meeting. label Nov 11, 2018
@@ -0,0 +1,33 @@
# Open Source Maintenance: November 2018 Core Meeting
Copy link
Collaborator

Choose a reason for hiding this comment

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

I dont think we should call this "core". Can we call this "contributors" meeting ? We don't want to make this exclusive

Copy link
Member Author

Choose a reason for hiding this comment

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

We always called them Core meeting (check the other notes) - and they are exclusive, since participation is limited to the developers in the Slack 🤷‍♂️

Copy link
Member Author

Choose a reason for hiding this comment

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

And the paragraph right below this title clearly explains the nature of the core, so I don't see any issue with this wording.

Copy link
Member Author

Choose a reason for hiding this comment

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

That said, what could be done is:

  1. come up with a clearer set of rules that defines what is a "core" contributor (good reference: https://www.youtube.com/watch?v=376dz1oIr3g)
  2. reword across the whole repo this concept, reflecting what decided in point 1

Copy link
Member Author

Choose a reason for hiding this comment

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

I've also created a dedicated discussion for this -> #59

core-meetings/2019-11-oss-meeting.md Show resolved Hide resolved
core-meetings/2019-11-oss-meeting.md Show resolved Hide resolved
core-meetings/2019-11-oss-meeting.md Outdated Show resolved Hide resolved
@axe-fb
Copy link
Collaborator

axe-fb commented Nov 16, 2018

I think we are good to publish these notes.

3. The [Releases.md](https://github.com/facebook/react-native/blob/master/Releases.md) file will be updated to reflect the current set of steps necessary for a release
4. For cherry pick requests, in case of complexities emerging during the release process, the author of the request will be asked to create a PR towards the branch
5. The FB team will attempt to be more empathetic when landing commits like React syncs to prevent issues
6. Another maintenance meeting will happen before end of the year, and among other things we'll discuss about merging the two GitHub organizations React Community and React Native Community and we'll create a shared guideline for how to maintain repos in them
Copy link
Member

Choose a reason for hiding this comment

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

Can we get some clarification on this point? I mean it's been probably all said - it's just that I didn't take part in the meeting.

1/ What is the motivation to merge them?
2/ Is the React Community going to stay, but be renamed to e.g. Expo Community where Expo-related modules are going to stay?

CC: @brentvatne, you might be the person to answer this more in details! Thank you :)

Copy link
Member Author

Choose a reason for hiding this comment

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

This has not been discussed, that line is basically a "homework for next time" thing - I'll word it better. But overall the sentiment is that it would allow better understanding and management and stability of the components maintained in there.

Copy link
Member Author

Choose a reason for hiding this comment

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

I've changed the wording to make this more clear and less like, we are taking decisions on this now.

@kelset kelset merged commit 610230f into master Nov 22, 2018
@kelset kelset deleted the notes-about-nov-meeting branch November 22, 2018 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📑 Meeting This indicates an issue/PR related to a core meeting.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants