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

Support for search-by-language #222

Closed
wants to merge 19 commits into from
Closed

Support for search-by-language #222

wants to merge 19 commits into from

Conversation

novalis
Copy link

@novalis novalis commented Jun 7, 2013

Not the world's finest C code, but does the job. Fixes #128

@novalis
Copy link
Author

novalis commented Jun 7, 2013

(actually, I just amended that commit because I realized I didn't need to specifically list the languages in options.c)

@novalis
Copy link
Author

novalis commented Jun 8, 2013

(er, please disregard; the previous commit was, unfortunately, correct)

@novalis
Copy link
Author

novalis commented Jun 14, 2013

OK, I think I'm done fixing bugs and adding features for now.

@aehlke
Copy link

aehlke commented Aug 2, 2013

How is this going? Any way I could help? This is the biggest thing keeping me from switching entirely over to ag from ack. Thanks!

@novalis
Copy link
Author

novalis commented Aug 2, 2013

Which part is the biggest thing? Search-by-language works on my branch, and there are a few other bugs fixed as well. It's just that my branch is slower because it actually correctly handles gitignores.

@aehlke
Copy link

aehlke commented Aug 3, 2013

I meant just being able to specify --py for example is a great asset of ack. I'll try your branch out, thanks. If the maintainer isn't up to it anymore perhaps the community could step up with its own canonical fork.

Sent from my iPhone

On Aug 2, 2013, at 4:39 PM, novalis notifications@github.com wrote:

Which part is the biggest thing? Search-by-language works on my branch, and there are a few other bugs fixed as well. It's just that my branch is slower because it actually correctly handles gitignores.


Reply to this email directly or view it on GitHub.

@derekleverenz
Copy link

I'd like to see this merged or this feature added otherwise. Or maybe change the brew formula to use this fork.

@ggreer
Copy link
Owner

ggreer commented Oct 26, 2013

Sorry for the neglect. I've been pretty busy. :(

I'm going to do a coding session on October 26th at 1900UTC (noon pacific, or in 18 hours). I'll do my best to add this feature then. I don't think I'll merge everything in this pull request though, since it changes quite a few behaviors. I am aware of the gitignore issue, but I think most users value speed over correctness in edge cases.

If you want to follow along and give feedback, go to https://floobits.com/r/ggreer/the_silver_searcher/ at the specified time.

@novalis
Copy link
Author

novalis commented Oct 26, 2013

Thanks for the amazing offer. If I get home in time, I will try to join you, but it is unlikely that I will make it home before 2000UTC.

I think the gitignore stuff could (and should) be done more efficiently with caching.

@derekleverenz
Copy link

Oh dang I wasn't able to do that. Thanks for the update though, that's great.

@ggreer
Copy link
Owner

ggreer commented Oct 28, 2013

That certainly won't be the last real-time coding session. If you let me know some times that work for you, I might be able to schedule the next one accordingly. I'm on the west coast, and I'm usually awake from 8AM-midnight. That's 1300-700UTC.

@brondsem
Copy link

Seems like #283 has implemented this?

@ggreer
Copy link
Owner

ggreer commented Nov 27, 2016

Sorry for neglecting your work, but the main feature in this PR (language options) has been around in ag for a while. The git behavior isn't perfect, but it's also improved. Considering the size of this PR and the number of conflicts, all I can do is close it and apologize. Sorry.

@ggreer ggreer closed this Nov 27, 2016
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.

Question: Plans for limiting search to file types?
6 participants