Skip to content
This repository was archived by the owner on Apr 1, 2022. It is now read-only.

add Contributors yaml file + Make first PR flow #43

Merged
merged 2 commits into from
Jul 24, 2020

Conversation

aboullaite
Copy link
Contributor

This PR is related to #37! it adds the yaml file + a flow to let the bot guide users to make their contribution

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 23, 2020
@aboullaite aboullaite force-pushed the add-yaml-contributors branch 2 times, most recently from 534fb74 to cc5c5a1 Compare July 23, 2020 21:56
Signed-off-by: Mohammed Aboullaite <[email protected]>

fix format issues

Signed-off-by: Mohammed Aboullaite <[email protected]>

fix format issues

Signed-off-by: Mohammed Aboullaite <[email protected]>

fix format issues

Signed-off-by: Mohammed Aboullaite <[email protected]>

fix format issues

Signed-off-by: Mohammed Aboullaite <[email protected]>

fix format issues

Signed-off-by: Mohammed Aboullaite <[email protected]>
@aboullaite aboullaite force-pushed the add-yaml-contributors branch from 5c811b4 to 967fe73 Compare July 23, 2020 22:05
Signed-off-by: Mohammed Aboullaite <[email protected]>
Copy link
Contributor

@elinguiuriel elinguiuriel left a comment

Choose a reason for hiding this comment

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

Please can you push your PR in the develop branch ? without the translation the code is not ready for production it will break the french experience !
We need to wait your other PR on french translation before merging this PR.
This issue #38 also have to be resolve before merging

@aboullaite
Copy link
Contributor Author

I do not agree that we should block this PR because of french translation! The first language of the bot is english, and other translation are of course welcomed, but ad far as they don't block adding features! This PR will allow many first time contributors to make their first open source contribution, which is the number one goal of this project! Also, It's one of the important features that we should highlight for the promotion next Tuesday!

@aboullaite aboullaite merged commit 46adbfd into master Jul 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants