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

Add missing implementation of crypto-js ciphers #9191

Merged
merged 1 commit into from
May 5, 2016

Conversation

misak113
Copy link
Contributor

@misak113 misak113 commented May 2, 2016

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.
  • there ware uncoverred typings for cipher of crypto-js
  • was updated version number of crypto-js
  • added test cases crypto-js incorrect typings #9133

* there ware uncoverred typings for cipher of crypto-js
* was updated version number of crypto-js
* added test cases DefinitelyTyped#9133
@dt-bot
Copy link
Member

dt-bot commented May 2, 2016

crypto-js/crypto-js.d.ts

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

Checklist

  • pass the Travis CI test?

@misak113 misak113 mentioned this pull request May 2, 2016
4 tasks
@misak113
Copy link
Contributor Author

misak113 commented May 2, 2016

👍

@dragGH102
Copy link

+1. I really need the correct typings implementation to use it in my Angular 2 - Typescript project

@vvakame vvakame merged commit 9143f12 into DefinitelyTyped:master May 5, 2016
@vvakame
Copy link
Member

vvakame commented May 5, 2016

@misak113 thanks a lot!

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