Skip to content

Add info for first time contributors#2265

Merged
marvinhagemeister merged 1 commit into
masterfrom
contribution-guide
Jan 21, 2020
Merged

Add info for first time contributors#2265
marvinhagemeister merged 1 commit into
masterfrom
contribution-guide

Conversation

@marvinhagemeister

Copy link
Copy Markdown
Member

This PR adds a lot of missing information on how to contribute to Preact in CONTRIBUTING.md 👍 Based on the feedback from this excellent deep dive on youtube into our source code https://www.youtube.com/watch?v=0Uv6p-Etyqk 🎉

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 97.745% when pulling b396a09 on contribution-guide into e3591cc on master.

@prateekbh

Copy link
Copy Markdown
Member

Pavithra would be really glad about the impact!!!

@JoviDeCroock JoviDeCroock left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Awesome writeup Marvin!

@marvinhagemeister

Copy link
Copy Markdown
Member Author

Commented on her tweet just minutes ago! Hope she sees it 🤞

@marvinhagemeister marvinhagemeister merged commit 4cb1119 into master Jan 21, 2020
@marvinhagemeister marvinhagemeister deleted the contribution-guide branch January 21, 2020 19:55
@jamesb3ll

Copy link
Copy Markdown
Contributor

Great stuff! Although a lot was intuitive when I made a PR, this write up is great for diving into your first PR 😃

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.

5 participants