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

Contributor Onboarding (First chapter or two) #12

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Conversation

makew0rld
Copy link
Contributor

No description provided.

src/onboarding/github.md Outdated Show resolved Hide resolved
src/onboarding/github.md Outdated Show resolved Hide resolved
src/onboarding/github.md Outdated Show resolved Hide resolved
src/onboarding/github.md Outdated Show resolved Hide resolved
src/onboarding/github.md Outdated Show resolved Hide resolved
src/onboarding/github.md Show resolved Hide resolved
src/onboarding/github.md Outdated Show resolved Hide resolved
src/onboarding/github.md Outdated Show resolved Hide resolved
src/onboarding/github.md Outdated Show resolved Hide resolved
@darkdrgn2k
Copy link
Contributor

Not sure if this document is general github on boarding or tomesh specific

If tomesh specific i think there should be mention that issues are being used by us as tasks, which is NOT the normal purpose of them.

Also the issue template is custom as well.

@makew0rld
Copy link
Contributor Author

makew0rld commented Aug 31, 2020

It is general to GitHub, but because I'm trying provide the information a new contributor will need to work with Toronto Mesh.

  • If tomesh specific i think there should be mention that issues are being used by us as tasks, which is NOT the normal purpose of them.

  • Also the issue template is custom as well.

I will add info on both of these.

src/onboarding/github.md Outdated Show resolved Hide resolved
src/onboarding/github.md Outdated Show resolved Hide resolved
Co-authored-by: darkdrgn2k <[email protected]>
@darkdrgn2k darkdrgn2k self-requested a review September 7, 2020 12:56
darkdrgn2k
darkdrgn2k previously approved these changes Sep 7, 2020
Copy link
Contributor

@darkdrgn2k darkdrgn2k left a comment

Choose a reason for hiding this comment

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

I think its good enough for first run!

@makew0rld makew0rld marked this pull request as ready for review September 7, 2020 15:33
@makew0rld
Copy link
Contributor Author

@darkdrgn2k Need a re-review after resolving merge conflicts, thanks.

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.

3 participants