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

Add Peter Krevenets to Contributors list #322

Merged

Conversation

GuRuGuMaWaRu
Copy link

No description provided.

@Roshanjossey
Copy link
Member

@GuRuGuMaWaRu, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?

@Roshanjossey Roshanjossey merged commit aa90bd1 into firstcontributions:master Jul 24, 2017
@GuRuGuMaWaRu GuRuGuMaWaRu deleted the add-peter-krevenets branch July 24, 2017 05:29
@GuRuGuMaWaRu
Copy link
Author

@Roshanjossey, there is literally no journey to talk about as I'm yet to start it, but I'm grateful for this opportunity to try and make a PR - it actually felt good, even though it is a small thing!
I think it is a really good idea to implement version control in learning environment - even thought how wonderful it would be if all code challenge resources (like codewars, leet etc.) required making PRs to solve problems.

@Roshanjossey
Copy link
Member

@GuRuGuMaWaRu: I agree version control is an awesome tool to have around when you're learning.
It gives you a lot of flexibility and freedom to explore.
I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://firstcontributions.herokuapp.com

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.

2 participants