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

typeahead.js update #4772

Closed
wants to merge 1 commit into from
Closed

Conversation

Seltzer
Copy link
Contributor

@Seltzer Seltzer commented Jun 30, 2015

Major changes:

  • Updated bindings/tests/docs to deal with typeahead rewrite between v0.10.4 and v0.11.1
  • Previously, only Bloodhound types were generic. Now other typeahead types are too. It means we can replace with in many places.
  • Fixed Bloodhound methods so that they return Bloodhound rather than void.
  • Added typeahead examples as tests
  • Added Bloodhound tests
  • Marked unofficial functions as such
  • Fixed inconsistencies with commenting, braces, casing and whitespace across all files

Sorry about the nasty diff.

* Updated bindings/tests/docs to deal with typeahead rewrite between v0.10.4 and v0.11.1
* Previously, only Bloodhound types were generic. Now other typeahead types are too. It means we can replace any with TDatum in many places.
* Fixed Bloodhound methods so that they return Bloodhound<TDatum> rather than void.
* Added typeahead examples as tests
* Added Bloodhound tests
* Marked unofficial functions as such
* Fixed inconsistencies with commenting, braces, casing and whitespace across all files
@horiuchi
Copy link
Member

horiuchi commented Jul 1, 2015

@igochkov @gjunge Please review this pull request, and 👍 or 👎 ?

@RyanCavanaugh
Copy link
Member

Hello and thank you for your contribution!

Due to an excessively long queue of pull requests that have become stale, we are "declaring bankruptcy" and closing all PRs opened before May 1, 2016. If you'd still like to merge this code in, please open a new PR that has been merged and rebased with the master branch.

Going forward, we are committing to review or merge all PRs on a regular basis so this bankruptcy will not occur again. We apologize for the incovenience and hope you will continue to contribute to DefinitelyTyped in the future.

Thanks
The TypeScript and DefinitelyTyped teams

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.

None yet

3 participants