Skip to content

Fixes #10684 - #10751

Merged
vvakame merged 1 commit into
DefinitelyTyped:masterfrom
LOZORD:master
Sep 14, 2016
Merged

vvakame merged 1 commit into
DefinitelyTyped:masterfrom
LOZORD:master

Conversation

@LOZORD

@LOZORD LOZORD commented Aug 21, 2016

Copy link
Copy Markdown
Contributor

case 1. Add a new type definition.

  • checked compilation succeeds with --target es6 and --noImplicitAny options.
  • has correct naming convention
  • has a test file with the suffix of -tests.ts or -tests.tsx.

case 2. Improvement to existing type definition.

  • documentation or source code reference which provides context for the suggested changes. url http://api.jquery.com/html .
    • it has been reviewed by a DefinitelyTyped member.

I kept the deprecated pick declarations, but added the proposed (JSDoc-influenced) deprecation annotation to them. The added pickone and pickset declarations have identical types to their respective pick flavors.

I kept the deprecated `pick` declarations, but added the proposed (JSDoc-influenced) [deprecation annotation](microsoft/TypeScript#390) to them. The added `pickone` and `pickset` declarations have identical types to their respective `pick` flavors.
@dt-bot

dt-bot commented Aug 21, 2016 •

Copy link
Copy Markdown
Member

chance/chance.d.ts

to author (@cbowdon). Could you review this PR?
👍 or 👎?

Checklist

  • pass the Travis CI test?

@DxCx

DxCx commented Aug 24, 2016

Copy link
Copy Markdown
Contributor

Thanks @LOZORD, I was just about to do the same fix :)

@LOZORD

LOZORD commented Sep 13, 2016

Copy link
Copy Markdown
Contributor Author

@vvakame can this be merged yet?

@vvakame
vvakame merged commit b73aaa3 into DefinitelyTyped:master Sep 14, 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.

4 participants