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

[RFC] Add partner application process to ECOSYSTEM.md #37689

Closed
wants to merge 2 commits into from

Conversation

rickhanlonii
Copy link
Member

@rickhanlonii rickhanlonii commented Jun 5, 2023

Overview

This PR updates the ECOSYSTEM.md with more information for how to apply to be a partner, and includes an update to the joining process including a 3 month incubation period.

It also emphasizes that being a partner is a commitment of resources to improve React Native, and not a status symbol.

Changelog: internal

## Overview

This PR updates the ECOSYSTEM.md with more information for how to apply to be a partner, and includes an update to the joining process including a 3 month incubation period.

It also emphasizes that being a partner is a commitment of resources to improve React Native, and not a status symbol.
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Jun 5, 2023
@github-actions
Copy link

github-actions bot commented Jun 5, 2023

Fails
🚫

📋 Verify Changelog Format - See Changelog format

Warnings
⚠️ 📋 Missing Summary - Can you add a Summary? To do so, add a "## Summary" section to your PR description. This is a good place to explain the motivation for making this change.
⚠️ 📋 Missing Test Plan - Can you add a Test Plan? To do so, add a "## Test Plan" section to your PR description. A Test Plan lets us know how these changes were tested.

Generated by 🚫 dangerJS against 009f951

@rickhanlonii rickhanlonii changed the title Add partner application process to ECOSYSTEM.md [RFC] Add partner application process to ECOSYSTEM.md Jun 5, 2023
@analysis-bot
Copy link

analysis-bot commented Jun 5, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,744,350 -225,902
android hermes armeabi-v7a 8,056,503 -174,945
android hermes x86 9,235,453 -248,466
android hermes x86_64 9,086,268 -240,123
android jsc arm64-v8a 9,307,098 -224,016
android jsc armeabi-v7a 8,496,599 -173,042
android jsc x86 9,369,085 -246,578
android jsc x86_64 9,623,994 -238,237

Base commit: 0201e51
Branch: main

* Contributing to the release process. Examples include being a [community releaser](https://reactnative.dev/contributing/release-roles-responsibilities#release-role-2-community-releaser), testing new releases, technical support for release issues.
* Engage on the core contributor Discord
### Application process
To become a React Native partner, an existing partner needs to refer and champion your application. Partners will undergo a 3 month incubating period, after which Partners will vote to convert to full membership.
Copy link
Collaborator

Choose a reason for hiding this comment

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

an existing partner needs to refer and champion your application

I don't think this aspect was discussed/mentioned during the meeting?

Copy link
Contributor

Choose a reason for hiding this comment

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

I do recall us talking about partners referring others, what are your concerns around this? Feel free to message the group about this too

Comment on lines +22 to +25
Maintaining partner status requires consistently meeting the baseline responsibilities, including:
* Attending monthly meeting.
* Contributing to the release process.
* Engaging in the core contributor Discord.
Copy link
Collaborator

@kelset kelset Jun 8, 2023

Choose a reason for hiding this comment

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

Suggested change
Maintaining partner status requires consistently meeting the baseline responsibilities, including:
* Attending monthly meeting.
* Contributing to the release process.
* Engaging in the core contributor Discord.
Maintaining partner status requires consistently meeting the baseline responsibilities, including:
* Attending monthly meeting.
* Contributing to the release process.
* Engaging in the core contributor Discord.
* Meaningfully contribute to the ecosystem and the broader community

since most existing partners do very important things (ex IR and Callstack and Expo+SWM doing the confs, Wix maintaining their libs) for the broader community it might be worth to mention? Like more partners do support the broader community than the ones contributing to the release process, so might be worth to have some sort of mention there for perspective partners to consider themselves "reaching the bar" for applying

Copy link
Collaborator

@alfonsocj alfonsocj Jun 15, 2023

Choose a reason for hiding this comment

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

I agree with @kelset. Maybe not all partners should contribute to the release process but have meaningful contributions to the ecosystem. I think a good idea is to add examples of meaningful contributions (doesn't need to be concrete) as the definition is very vague. We mentioned in the meeting that it could be code contributions to core and popular libraries but also conferences and podcasts.

Copy link
Member Author

Choose a reason for hiding this comment

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

Great points. I didn't change any of these here though, just changed the formatting, so let's followup to up if we want to change the baseline responsibilities.

### Incubating partners:
* There are no current incubating partners.

When you are contributing to React Native, you'll most likely meet somebody who works at one of the partner companies and who is a core contributor.
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: I'm not sure I understand why this phrase is here/what purpose it serves; is it to help perspective partners understand how they can find someone to champion them? 🤔

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 was there before; it feeds to the next section about core contributors and delineates the difference between Partner (company) and Contributor (individual).

* Contributing to the release process.
* Engaging in the core contributor Discord.

Examples of contributing to the release include being a [community releaser](https://reactnative.dev/contributing/release-roles-responsibilities#release-role-2-community-releaser), testing new releases, and technical support for release issues.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not totally related to this doc but I think is a good idea to clarify in the release role doc that becoming part of the release crew is a multi month commitment until the baton is passed to the next crew. Additionally, the release crew is responsible for all active releases in the support window, not just the upcoming one.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Comment on lines +22 to +25
Maintaining partner status requires consistently meeting the baseline responsibilities, including:
* Attending monthly meeting.
* Contributing to the release process.
* Engaging in the core contributor Discord.
Copy link
Collaborator

@alfonsocj alfonsocj Jun 15, 2023

Choose a reason for hiding this comment

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

I agree with @kelset. Maybe not all partners should contribute to the release process but have meaningful contributions to the ecosystem. I think a good idea is to add examples of meaningful contributions (doesn't need to be concrete) as the definition is very vague. We mentioned in the meeting that it could be code contributions to core and popular libraries but also conferences and podcasts.

@facebook-github-bot
Copy link
Contributor

@rickhanlonii has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@rickhanlonii has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@github-actions
Copy link

This pull request was successfully merged by @rickhanlonii in 9e1c3d2.

When will my fix make it into a release? | Upcoming Releases

@github-actions github-actions bot added the Merged This PR has been merged. label Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants