Skip to content
This repository has been archived by the owner on Sep 2, 2022. It is now read-only.

Small grammar fixes #4955

Merged
merged 1 commit into from
Dec 18, 2019
Merged

Small grammar fixes #4955

merged 1 commit into from
Dec 18, 2019

Conversation

robin-macpherson
Copy link
Contributor

Some more minor grammar fixes for the docs.

@robin-macpherson
Copy link
Contributor Author

robin-macpherson commented Dec 15, 2019

@nikolasburk Just pasting my response from my other merged PR (#4954 ) here in case you didn't see it:

Thank you @nikolasburk ! Would you mind also merging #4955 ? Also, what's the correct way for me to do this in the future so that I can add you as a reviewer and not have to tag you in a comment?

@robin-macpherson
Copy link
Contributor Author

robin-macpherson commented Dec 18, 2019

@do4gr @kuldar @jasonkuhrt @dpetrick is there anyone who can help me merge this PR and understand why both of my first two PRs got stuck like this? I'll be happy to make sure I do it correctly next time. Please see the above comment and linked PR for context - thank you! 🙇🏻‍♂️

@nikolasburk nikolasburk merged commit 2a38a5a into prisma:master Dec 18, 2019
@nikolasburk
Copy link
Member

Thanks a lot @robin-macpherson! 🙏 Sorry for the delay, this is just a capacity issue at the moment since we're focusing all our resources on Prisma 2 at the moment.

@robin-macpherson
Copy link
Contributor Author

No problem and thank you for helping me on both of these @nikolasburk ! So just to be clear, I did everything correctly and it was just capacity? I was worried that since there were no reviewers added to the PR perhaps nobody had been notified of it.

Keep up the great work with Prisma 2 and thanks for everything!

@nikolasburk
Copy link
Member

Yes, everything was good from your end! Feel free to keep pinging me on these kinds of PRs, I'm trying to get back as soon as possible! 🙂

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants