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

Ping @lildude from now on #3856

Merged
merged 1 commit into from
Oct 13, 2017
Merged

Ping @lildude from now on #3856

merged 1 commit into from
Oct 13, 2017

Conversation

lildude
Copy link
Member

@lildude lildude commented Oct 11, 2017

@bkeepers is no longer actively involved with Linguist so it doesn't makes sense to ping him anymore. Updating the docs to get peeps to ping me.

As an aside, the corpus gathering on the GitHub side doesn't work at the mo so the ping is a bit moot at the moment, but it is on my list to fix it 🔜

@Alhadis
Copy link
Collaborator

Alhadis commented Oct 11, 2017

What do you mean the corpus gathering doesn't work on GitHub's side? Is this referring to the way public search results can't be browsed using the NOT nothack hack unless you're logged into GitHub?

@lildude
Copy link
Member Author

lildude commented Oct 11, 2017

I'm referring to the thing we used to do where we created a big fat zip file and bunged it on S3 for peeps to download and test against. That part doesn't work. It used to take advantage of the ability to search public repos anonymously without limiting to a user/repo/org but the change that intro'd the auth or limiting props that borked it. The interface has also now been removed so I'll need to implement it some other way.

@Alhadis
Copy link
Collaborator

Alhadis commented Oct 11, 2017

Definitely a much better and efficient solution than my hacky harvester script. :( Which has to be operated semi-manually, and the downloading process needs to happen separately. It's all a colossal mess because of no API from GitHub that allows filetypes/filenames to be searched without specifying a user or organisation. 😢

@lildude
Copy link
Member Author

lildude commented Oct 13, 2017

@Alhadis or @pchaigno can one of your guys 👍 this so I can merge it. 🙇

@lildude lildude merged commit a47008e into master Oct 13, 2017
@lildude lildude deleted the lildude/ping-lildude branch October 13, 2017 16:49
@github-linguist github-linguist locked as resolved and limited conversation to collaborators Jun 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants