add swagger-sails-hook #13676

Merged
merged 3 commits into from Jan 2, 2017

Projects

None yet

3 participants

@micmro
Contributor
micmro commented Jan 2, 2017

Please fill in this template.

  • Make your PR against the master branch.
  • Use a meaningful title for the pull request. Include the name of the package modified.
  • Test the change in your own code. (Compile and run.)
  • Follow the advice from the readme.
  • Avoid common mistakes.
  • Run tsc without errors.
  • Run npm run lint package-name if a tslint.json is present.

Select one of these and delete the others:

If adding a new definition:

  • The package does not provide its own types, and you can not add them.
  • If this is for an NPM package, match the name. If not, do not conflict with the name of an NPM package.
  • Create it with npm run new-package package-name, not by basing it on an existing project.

If changing an existing definition:

  • Provide a URL to documentation or source code which provides context for the suggested changes: add test cases for sails and connect middleware
  • Increase the version number in the header if appropriate.
  • If you are making substantial changes, consider adding a tslint.json containing { "extends": "../tslint.json" }.

If removing a declaration:

  • If a package was never on DefinitelyTyped, you don't need to do anything. (If you wrote a package and provided types, you don't need to register it with us.)
  • Delete the package's directory.
  • Add it to notNeededPackages.json.
@dt-bot
Member
dt-bot commented Jan 2, 2017

swagger-sails-hook/index.d.ts

Checklist

@micmro micmro changed the title from Swagger sails hook to add swagger-sails-hook Jan 2, 2017
@micmro
Contributor
micmro commented Jan 2, 2017 edited

This is the last remaining one of the swagger-node-runner middleware:
swagger-sails-hook source file: https://github.com/apigee-127/swagger-sails/blob/master/lib/index.js

swagger-sails-hook/index.d.ts
+ * @see {@link http://sailsjs.com/documentation/concepts/extending-sails/hooks/hook-specification|Sails Hook Docs}
+ * @see {@link http://sailsjs.com/documentation/anatomy/api/hooks/my-hook/index-js|Sails Hook Example}
+ */
+ export interface SailsHook {
@andy-ms
andy-ms Jan 2, 2017 Contributor

Indent body of interface

@andy-ms andy-ms merged commit 0c56e65 into DefinitelyTyped:master Jan 2, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@micmro micmro deleted the micmro:swagger-sails-hook branch Jan 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment