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

Added ability to search for blocknames and ids like OAB used to have #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Denevien
Copy link

@Denevien Denevien commented Dec 2, 2013

No description provided.

Uses .search SEARCHTYPE INPUT

Searchtypes: "name" and "id". 

Inputs: "name" - 1 word keyword: uses the block name to return a list of block names that contain the keyword and the id for that block.
"id" - can be "###" or "###:###": uses the block ID to return the block name.
@edk0
Copy link
Contributor

edk0 commented Dec 2, 2013

Thoughts: not async and file doesn't need to be reopened many times anyway; id based search can be much more efficient; no need to be told a search type when we could figure it out from the search term.

@Denevien
Copy link
Author

Denevien commented Dec 2, 2013

Thanks for the feedback, this is my first real attempt at python, so wanted
to give it a go. I'll toy with it some more, and try to make it more
efficient.
On Dec 2, 2013 2:00 AM, "Ed Kellett" [email protected] wrote:

Thoughts: not async and file doesn't need to be reopened many times
anyway; id based search can be much more efficient; no need to be told a
search type when we could figure it out from the search term.


Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-29600083
.

…rent id

search id 3 now properly returns both dirt and podzol, instead of just the podzol.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants