feat(Entity) Deprecate addAll and rename it to setAll

[elvisun]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/ngrx/platform/blob/master/CONTRIBUTING.md#commit
• Tests for the changes have been added (for bug fixes / features)
• Documentation has been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

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

What is the current behavior?

Currently addAll removes all existing entities and then add all, which is very confusing and bug-prone.
Closes #2330

What is the new behavior?

Deprecated addAll and added an identical setAll

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

[alex-okrushko]

I think you should rename addAllMutalby to setAllMutable and just point addAll and setAll to createStateOperator(setAllMutably).

[elvisun]

done

[ngrxbot]

Preview docs changes for 2ecd958 at https://previews.ngrx.io/pr2348-2ecd958/

[timdeschryver]

We should also change the schematics and @ngrx/data (can be done in separate PRs?)

Schematics:

• https://github.com/ngrx/platform/blob/master/modules/schematics/src/entity/files/__name%40dasherize%40if-flat__/__name%40dasherize%40group-reducers__.reducer.ts.template#L55

• https://github.com/ngrx/platform/blob/master/modules/schematics/src/entity/creator-files/__name%40dasherize%40if-flat__/__name%40dasherize%40group-reducers__.reducer.ts.template

Data:

• https://github.com/ngrx/platform/blob/master/modules/data/src/reducers/entity-collection-reducer-methods.ts

[timdeschryver]

LGTM

[brandonroberts]

Thanks @elvisun!