-
-
Notifications
You must be signed in to change notification settings - Fork 381
Remove first prefix from submenus in library reference #827
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
Conversation
JinShil
commented
Jan 24, 2015
I think we should move forward with this, but just to make sure I've asked around. http://forum.dlang.org/post/ma0kuu$1f08$1@digitalmars.com |
@andralex If it's not too hard, please use the new "Permalink" links (by right-clicking them) to share forum URLs. http://forum.dlang.org/help#canonical |
Yes, please! What about |
looks like this is good to go. please rebase |
I rebased, but I have no idea if I did it right. It looks like everything's fine to my untrained eye, but please apply scrutiny. |
LIBITEM_RECURSE2=.​$1$(LIBITEM_RECURSE2 $+) | ||
>>>>>>> 870183cbd442b6af8e57ab2cdbab1ce0659ee586 | ||
_= |
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.
Nope, you need to resove this merge conflict.
I agree, but that looks like a non-trivial change. If I can get to it, I'll submit a separate pull request. |
I have an idea on how to handle it. |
Ok, I hope we're good to go now. |
Please rebase, there is a merge commit in your commit list :)
|
Actually, not every commit. Leave the commits containing your changes, delete the other lines. |
screw it! I'm making a new pull request. |
You could have just recreated the branch... |
Redo #827: Remove first prefix from submenus in library reference
|