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

feat(completion): Add blink-cmp-git for blink.cmp #864

Merged
merged 5 commits into from
Feb 19, 2025

Conversation

MuntasirSZN
Copy link
Contributor

@MuntasirSZN MuntasirSZN commented Feb 17, 2025

Describe what this PR does / why we need it

Blink.cmp now has its own git source, named blink-cmp-git. This pr adds it to the docs.

Does this pull request fix one issue?

None

Describe how you did it

Well, just rewrote the doc.txt and README.md.

Describe how to verify it

See diff

Special notes for reviews

Checklist

  • Passing tests and linting standards
  • Documentation updates in README.md and doc/octo.txt

@wd60622 wd60622 added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Feb 17, 2025
Copy link
Collaborator

@wd60622 wd60622 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small suggestion

@MuntasirSZN
Copy link
Contributor Author

@wd60622 Now fixed.

Copy link
Collaborator

@wd60622 wd60622 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the edit!

@wd60622 wd60622 merged commit a8cc591 into pwntester:master Feb 19, 2025
3 checks passed
@MuntasirSZN
Copy link
Contributor Author

@wd60622 Should we add docs for emoji completion using cmp-emoji and blink-cmp-emoji? They provide completion using markdown syntax.

@wd60622
Copy link
Collaborator

wd60622 commented Feb 19, 2025

@wd60622 Should we add docs for emoji completion using cmp-emoji and blink-cmp-emoji? They provide completion using markdown syntax.

Sure

@MuntasirSZN
Copy link
Contributor Author

Pr incoming!

@MuntasirSZN MuntasirSZN mentioned this pull request Feb 20, 2025
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants