feat(contribution-types): adds mentoring

[mloning]

What:
Add support for mentoring (see all-contributors/all-contributors#423)

Why:

How:
Added mentoring to src/tasks/processIssueComment/utils/parse-comment/index.js

Checklist:

• Documentation
• Tests
• Ready to be merged
• Added myself to contributors table

Comments:
If tests are needed, could you point me to the files that I have to change/add?

[mloning]

Seems like the test_and_deploy check got stuck? Is there anything else I need to do?

[mloning]

@all-contributors please add @mloning for code

[allcontributors]

@mloning

I've put up a pull request to add @mloning! 🎉

[omarcostahamido]

bump

[Berkmann18]

Hopefully, the required check will stop hanging.

[omarcostahamido]

bump

(sorry for doing it again, but it has been a while...)

I'll do another PR if it's necessary (#321 ), just let me(us) know what we can do to get this merged.

Thanks!

[Berkmann18]

@jakebolam Any ideas as to why this CI issue happens?

[vercel]

@Berkmann18 is attempting to deploy a commit to the All Contributors Team on Vercel.

A member of the Team first needs to authorize it.

[omarcostahamido]

eeek
why is it so hard to fix this?

[omarcostahamido]

ok, now it is giving some more insightful information: conflicting files.

src/tasks/processIssueComment/utils/parse-comment/index.js
yarn.lock

@mloning can you fix the PR branch?
@Berkmann18 as soon as he does that, can you merge it?

thx

[mloning]

@omarcostahamido thanks for looking into this. Not sure what I have to do, git tells me the files I changed were deleted on master? Have they been moved?

[Berkmann18]

The whole project had an overhaul, and the file where you changes would essentially move to is: https://github.com/all-contributors/app/blob/master/lib/parse-comment.js
@mloning So if you port your changes to that file (after rebasing), the PR should be mergeable. Apologies for the delay.

I was hoping to have documentation around the new architecture that will affect the other repos but life got on the way.

[mloning]

@omarcostahamido @Berkmann18 alright I cleaned it up - anything else still missing or is this good to go?

[Berkmann18]

Awesome! Thanks a lot!

[Berkmann18]

@gr2m I see that Vercel is having an auth problem, is there anything we can do to fix that?
If not I'll merge it.

[vercel]

Deployment failed with the following error:

The most recent charge for your active payment method has failed. Please update it here: https://vercel.com/teams/all-contributors-app/settings/billing.

[gr2m]

@gr2m I see that Vercel is having an auth problem, is there anything we can do to fix that?
If not I'll merge it.

I had to authorize the deploy because the PR is from a fork. I should be able to invite to the Vercel org, but it's $20/month, that I'm currently paying. Okay if I do the approvals for now, until we either get sponsoring from Vercel, or move to begin.com for deployments (which is something I'd like to do anyway)

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/all-contributors-app/app/rohwwzvkr
✅ Preview: https://app-git-fork-mloning-mentoringemoji.all-contributors-app.vercel.app

[gr2m]

You can install https://github.com/apps/all-contributors-preview for testing the change. Make sure you don't have https://github.com/apps/allcontributors installed on the same repository, the two might conflict with each other

[gr2m]

looks good to me:
gr2m/sandbox#163 (comment)

[github-actions]

🎉 This PR is included in version 1.12.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[omarcostahamido]

omg, we did it, at last!
thank you @mloning @Berkmann18 @gr2m
🎉