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

Fix typos at hooks.asc #107

Merged
merged 1 commit into from
Jan 13, 2022
Merged

Fix typos at hooks.asc #107

merged 1 commit into from
Jan 13, 2022

Conversation

zettaittenani
Copy link
Contributor

8.3 Git のカスタマイズ - Git フックのページにて、2点 typo と思われる箇所を見つけましたので、
修正させていただきたいです。

issue: #106

@tnir
Copy link
Collaborator

tnir commented Mar 1, 2020

@harupong Could you take a look?

@okuryu
Copy link
Collaborator

okuryu commented Mar 1, 2020

@harupong doesn't have enough time to work on this project, so I'm talking to him to promote me to a maintainer for this project. I will merge this PR as soon as the permissions are updated.

@zettaittenani
Copy link
Contributor Author

@okuryu
I know you're very busy, but I’d be grateful if I could know the status.

I'm talking to him to promote me to a maintainer for this project. I will merge this PR as soon as the permissions are updated.

@zettaittenani
Copy link
Contributor Author

zettaittenani commented Jan 13, 2022

@tnir @okuryu @kenjis
Hello, excuse me for mentioning multiple people.
Could you guys merge this approved PR? Or I'll close it in 10 days.
Thank you in advance.

@tnir
Copy link
Collaborator

tnir commented Jan 13, 2022

@zettaittenani Thanks for reminding repo owners. I do not have write permission yet so consult @okuryu @kenjis to get this merged 😭

@tnir
Copy link
Collaborator

tnir commented Jan 13, 2022

@jnavila Any comments on merging unmerged PRs like this? YOU ARE the only person who merged in the past two years 😁

@kenjis
Copy link

kenjis commented Jan 13, 2022

@zettaittenani I also don't have write permission.

@jnavila
Copy link
Member

jnavila commented Jan 13, 2022

Who's ready to take the maintainership of this project?

@jnavila jnavila merged commit a3464cf into progit:master Jan 13, 2022
@tnir
Copy link
Collaborator

tnir commented Jan 13, 2022

@jnavila Thanks for merging. I would maintain the repo if you add me to the repo 😬

@zettaittenani
Copy link
Contributor Author

zettaittenani commented Jan 13, 2022

Thank you guys so much for the quick confirmation and decision!

@okuryu
Copy link
Collaborator

okuryu commented Jan 13, 2022

@jnavila Thanks for the merge. I will continue to maintain this repo in the future, but I am still waiting for @harupong to update the permissions.

@jnavila
Copy link
Member

jnavila commented Jan 13, 2022

I'll give you permissions.

@tnir tnir mentioned this pull request Jan 13, 2022
@tnir tnir added the ch08 label Jan 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants