feat(recommended)!: add four rules to recommended #56
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
BREAKING CHANGE: RxJS ^7.2.0 is required.
BREAKING CHANGE:
recommended
configuration had 4 rules added to it. See full changelog for details.Resolves #35 .
no-topromise
:toPromise()
was deprecated by RxJS, so this will be very useful for helping developers migrate to one of the two manual editor suggestions (lastValueFrom
orfirstValueFrom
).prefer-observer
: RxJS deprecated non-observer object subscribes, so this will help migrate.prefer-root-operators
: RxJS moved operators to root in 7.2.0 and deprecated the old site, so this will help migrate.throw-error
: This rule will rarely be triggered because it was recently changed to only apply to the static creation functionthrowError()
. But in those rare cases, it should always be enforced, similar tono-implicit-any-catch
.