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.

Sign up for GitHub

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

Jump to bottom

搜索功能 #6

Merged
merged 1 commit into from
Jun 16, 2021
Merged

搜索功能 #6

merged 1 commit into from
Jun 16, 2021

Conversation

zoroqi
Copy link
Contributor

@zoroqi zoroqi commented Jun 14, 2021

No description provided.

@zoroqi zoroqi mentioned this pull request Jun 14, 2021
Closed
chenjiandongx
chenjiandongx reviewed Jun 16, 2021
View reviewed changes
Copy link
Owner

@chenjiandongx chenjiandongx left a comment •
edited

Choose a reason for hiding this comment

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

@zoroqi

我觉得这个 PR 挺好的，思考了一下 -d 的用法，我猜你那边应该是 把命令的 md 文件放在了 /command 目录下吧，每次可能都要使用 -d 来指定就显得比较麻烦。

所以我想的另外一个解决方案就是把这个指定路径可以通过 pls 的子命令来编辑，一劳永逸，我晚点在你的 PR 上做些改动。

@chenjiandongx chenjiandongx merged commit cb5a1a7 into chenjiandongx:master Jun 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chenjiandongx chenjiandongx chenjiandongx left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@zoroqi @chenjiandongx