[Documentation] Express Typing

It’s really good to see a web framework that deals with TypeScript as a first class citizen! I’m really excited about this project because of this.

However, when turning the option: noImplicitAny in the tsconfig.json the compiler will produce warnings about a lot of the examples in the documentation.

I highly recommend adding Express TypeScript declarations to the examples like so:

import {Request,Response,NextFunction} from "@types/express";


@Controller({})
export class UsersController {
	getAllUsers(req:Request, res:Response, next:NextFunction) {
		// will have auto completion, type safety ...etc
	}
}

[marcus-sa]

Then the server package would still need to have both Fastify and Nest platform installed for it to work, and Nest still supports JS, so that’s not an option either, plus it’s more a downgrade than an actual bug fix imo. My advice? Stop being lazy.

[KerryRitter]

@ThomRick He’s asking that the examples on https://kamilmysliwiec.gitbooks.io/nest/content/ add the Express types. Right now they just say:

@Get('/:id')
public async getUser(@Response() res, @Param('id') id) {

as opposed to

@Get('/:id')
public async getUser(@Response() res: Response, @Param('id') id) {