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
Deduplicate command names in generated LSP support doc #10563
base: master
Are you sure you want to change the base?
Conversation
This is my contributions to the project for Issue #10316 |
xtask/Cargo.toml
Outdated
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.
This change should be dropped
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.
Why?
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.
Its not changing anything in the file, nor is it contributing to the change in the pr, therefore its best to remove the file from the commit/s. This only dirties histories of files.
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.
I rebased and fixup'd everything into 1 commit which drops the unneeded commits.
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.
It seems like those commits might have been squashed into the single commit instead - those whitespace changes are still in the diff
3dc1186
to
e464259
Compare
Ready for second peer review please. |
ready for another PR review. |
can I get another approver pls? |
Using unique from itertools::Itertools function to clear out and duplicate string entries. Tested on my ARM64 Windows 11 system with mdbook and generated via xtask docgen.