-
Notifications
You must be signed in to change notification settings - Fork 73
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
Recommend full URI to issues instead of just the number #19
Comments
@slashsbin not interested in this change? |
hi Actually, I don't totally agree with you, although I agree that doing so will improve terminal log viewing. Personally, for me, it's an overkill to paste the full URL because I need to copy/paste it from somewhere else beforehand. Another thing is, doing so may break automatcally closing issues using keywords. Maybe we should simplify browsing issues by other means like:
What do you think? |
It seems like GitLab is OK with automatic closing of issues by full URL. |
GitHub is also okay with automatic closing by full URL. I also found that I
almost never remember the issue number I'm working on so I need to
copy-paste it anyway. Especially when working with three-four digit issue
numbers.
…On Wed, Aug 23, 2017, 9:14 AM Mohammad Shokri ***@***.***> wrote:
It seems like GitLab is OK with automatic closing of issues by full URL
<https://docs.gitlab.com/ee/user/project/issues/automatic_issue_closing.html>
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#19 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABAuVAWvUrVnOHdCbADsJUqK-RGc99dqks5sa9FUgaJpZM4N-HE7>
.
|
hi |
Perfect, we can consider this issue closed, thank you! |
We always copy paste the entire URL to the issue in the last line instead of just #123.
Fixes https://github.com/slashsBin/styleguide-git-commit-message/issues/1
in the terminal while reading the git log.The text was updated successfully, but these errors were encountered: