-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
git-daemon: add page #5938
git-daemon: add page #5938
Conversation
This comment has been minimized.
This comment has been minimized.
fix another typo
huwh? @bl-ue xD |
Hey @Waples, why did you change that? The standard format is |
Owh derp yeah, I brainfarted xD |
Also @Waples since you have write access to the repo, you label PRs yourself. Also, you can like this PR to #3953 by editing this comment. |
I was busy with setting up another PR for Ah okay, I thought I should only add this PR to the main comment list when it was 'done'. Will add it now. |
Yeah, we add it as soon as the PR is open, because that way we can tell that someone is working on it. As soon as the PR is merged, anyone who is able to can tick the box (usually that's the PR merger). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the new page! I've left some comments below for you to review.
Co-authored-by: Starbeamrainbowlabs <[email protected]>
Co-authored-by: Starbeamrainbowlabs <[email protected]>
Co-authored-by: bl-ue <[email protected]>
Co-authored-by: marchersimon <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Waples
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @Waples!
common/
,linux/
, etc.)Ref: #3953