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

chore(): v8 major release [WIP] #6349

Merged
merged 145 commits into from
Jun 25, 2021
Merged

chore(): v8 major release [WIP] #6349

merged 145 commits into from
Jun 25, 2021

Conversation

kamilmysliwiec
Copy link
Member

@kamilmysliwiec kamilmysliwiec commented Feb 8, 2021

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Other... Please describe: Release

What is the current behavior?

N/A

What is the new behavior?

Features

Improvements

Bug fixes

Dependencies

Deprecations

  • common: deprecate HttpModule to remove hard dependency on the axios library. HttpModule is published as a new @nestjs/axios package now
  • core: deprecate the listenAsync() (use listen()) and startAllMicroservicesAsync() (use startAllMicroservices()) methods
  • microservices: deprecate the listenAsync() method (use listen() instead)

Does this PR introduce a breaking change?

[x] Yes
[ ] No

Other information

Shady Khalifa and others added 30 commits November 28, 2019 16:31
build: update package dependencies

fix: use bindAsync in server-grpc

test(): server gprc port already listen error

test(): remove surplus commit

feat: fix packages

# Conflicts:
#	package-lock.json
#	package.json
#	sample/04-grpc/package-lock.json
#	sample/04-grpc/package.json
New global prefix option to exclude some routes as a string or a RouteInfo
For class providers use a class reference instead of a class name as an injection token to address dependency injection conflicts when two classes share the same name.

resolve #5591
Added a method to set name property of exception object equal to class name which currently is always Error

Closes #5809
Added a method to set name property of exception object equal to class name which currently is always Error. Updated tests.

Closes #5809
Added a method to set name property of exception object equal to class name which currently is always Error. Updated tests.

Closes #5809
Add ability to extract a paramater from the MessageBody decorator.
@kamilmysliwiec
Copy link
Member Author

alpha.4 is published now :)

@MickL
Copy link
Contributor

MickL commented May 18, 2021

Alpha 4 is working fine for me with Nats 2.x :) @kamilmysliwiec I see there are new logs (Nats Ping, Nats reconnect). Is it possible to configure this logging? E.g. I would like to not see the ping in the logs (over days and weeks it will spam the logs) and I want to see only one time reconnect (currently it spams one per second which would result in massive logs after 1 hour downtime).

@gperdomor
Copy link
Contributor

@kamilmysliwiec to use v8 is enough to change the version of the packages in package.json or we need to do something else?

@kamilmysliwiec
Copy link
Member Author

@gperdomor there are a few minor breaking changes. You can find the migration guide & instructions on the 8.0.0 branch of the docs https://github.com/nestjs/docs.nestjs.com/tree/8.0.0

@gperdomor
Copy link
Contributor

@gperdomor there are a few minor breaking changes. You can find the migration guide & instructions on the 8.0.0 branch of the docs https://github.com/nestjs/docs.nestjs.com/tree/8.0.0

Thanks for the response, one last question, I notice that @nestjs/passport has no v8 version, the actual version is usable with the v8 alpha?

@kamilmysliwiec kamilmysliwiec mentioned this pull request Jun 24, 2021
2 tasks
@kamilmysliwiec kamilmysliwiec merged commit 72a88d5 into master Jun 25, 2021
@delete-merged-branch delete-merged-branch bot deleted the 8.0.0 branch June 25, 2021 12:52
@manjufy
Copy link

manjufy commented Jul 25, 2021

@gperdomor there are a few minor breaking changes. You can find the migration guide & instructions on the 8.0.0 branch of the docs https://github.com/nestjs/docs.nestjs.com/tree/8.0.0

This page https://github.com/nestjs/docs.nestjs.com/tree/8.0.0 seems to be broken.

Is there a reference guide I can refer to for move from v7.5 to v8

@wrslatz
Copy link

wrslatz commented Jul 25, 2021

Is there a reference guide I can refer to for move from v7.5 to v8

https://docs.nestjs.com/migration-guide

@manjufy
Copy link

manjufy commented Jul 25, 2021

Is there a reference guide I can refer to for move from v7.5 to v8

https://docs.nestjs.com/migration-guide

Thank you @wrslatz

@nestjs nestjs locked and limited conversation to collaborators Jul 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dependency injector resolves dependencies using a class name instead of a class reference