-
Notifications
You must be signed in to change notification settings - Fork 0
How to contribute
To ensure the quality of DefinitelyTyped repository, the typings must meet the following criteria:
- Definition files should be called
library.d.ts. In case there are multiple versions supported, the latest one will be without a version number in the file name. Older versions will be in the form oflibrary-1.0.d.ts.
- The typing must have a header with the following format:
// Type definitions for [LIBRARY NAME]
// Project: [LIBRARY URL]
// Definitions by: [AUTHOR NAME] <[AUTHOR URL]>
// Definitions: https://github.com/borisyankov/DefinitelyTyped
Example:
// Type definitions for Backbone 0.9.10
// Project: http://backbonejs.org/
// Definitions by: Boris Yankov <https://github.com/borisyankov/>
// Definitions: https://github.com/borisyankov/DefinitelyTyped
- The typing must be placed in a folder. The folder name must be similar to library name.
Example:
qunit.d.tsin in a folder namedqunit
- Take advantage of TypeScript JSDoc support.
- The typing must have tests. These tests must be in a files called
library-tests.ts. They are not runnable, in the TDD way. They contain code that should compile with no errors, usually taken from the documentation samples of the library.
Example:
backbone.d.tshas a test file namedbackbone-tests.tsHere you can see a real test file.
- Check your code. run
npm testbefore send pull request.- If test failed. try with
tsc --noImplicitAny your/code.d.ts. fix your code or addyour/code.d.ts.tscparams.
- If test failed. try with
If you need to use tscparams note that the contents of tscparams should be quoted. i.e. "" e.g. https://github.com/borisyankov/DefinitelyTyped/blob/master/ace/all-tests.ts.tscparams
- Be careful to use a module to avoid conflicts to your internal interfaces and the interfaces from another typings.
The
jQuery.bbqtyping has the interfaces in a module namedJQueryBbqExample:
module JQueryBbq {
interface JQuery {
//...
}
//...
}
interface JQueryStatic {
bbq: JQueryBbq.JQuery;
//...
}- Update the README file to put a reference to your definition with the following style:
* [LIBRARY NAME WITH LINK] (by [AUTHOR NAME WITH LINK])
Example:
This is general guidance to answer common questions.
You don't need to do this unless we request you to do so.
Initially make sure you have DT added as an upstream remote in your fork. Do this by :
git remote add upstream https://github.com/borisyankov/DefinitelyTyped.git
Now whenever you want the latest changes from DT added to your local branch just run these two commands :
git fetch upstream
# Fetches any new changes from DefinitelyTyped
git merge upstream/master
# Merges any changes fetched into your working files
Thanks for contributing!