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

[GSoD 2021] Added CONTRIBUTING.md #140

Merged
merged 7 commits into from
Sep 10, 2021
Merged

[GSoD 2021] Added CONTRIBUTING.md #140

merged 7 commits into from
Sep 10, 2021

Conversation

iamrajiv
Copy link
Member

@iamrajiv iamrajiv commented Jul 6, 2021

Signed-off-by: Rajiv Ranjan Singh <rajivperfect007@gmail.com>
@iamrajiv iamrajiv changed the title Added CONTRIBUTING.md [GSoD 2021] Added CONTRIBUTING.md Jul 6, 2021
@iamrajiv iamrajiv requested a review from huan July 11, 2021 01:30
Signed-off-by: Rajiv Ranjan Singh <rajivperfect007@gmail.com>
proudofsimin
proudofsimin previously approved these changes Jul 14, 2021
Copy link

@Rohitesh-Kumar-Jain Rohitesh-Kumar-Jain left a comment

Choose a reason for hiding this comment

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

pass CI test so that we can continue reviewing

Copy link
Member

@huan huan left a comment

Choose a reason for hiding this comment

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

Thanks for creating this CONTRIBUTING file!

As we have already put a contributing docs at our website https://wechaty.js.org/docs/contributing/ , so I'd like to suggest that we can move all the content to our website, and put the website link of https://wechaty.js.org/docs/contributing/ in this file.

Please feel free to improve the https://wechaty.js.org/docs/contributing/ as you need, and then change this file content to just link to our website.

Thank you very much.

@huan huan merged commit 0e7c83f into wechaty:main Sep 10, 2021
@huan
Copy link
Member

huan commented Sep 10, 2021

Thank you very much!

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

Successfully merging this pull request may close these issues.

4 participants