feat(core): add Collection.slice() method

[rubiin]

Adds support for Collection.slice()

[codecov]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (79af75c) 99.64% compared to head (400de2d) 99.64%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #4608   +/-   ##
=======================================
  Coverage   99.64%   99.64%           
=======================================
  Files         215      215           
  Lines       14247    14261   +14     
  Branches     3389     3393    +4     
=======================================
+ Hits        14196    14210   +14     
  Misses         50       50           
  Partials        1        1           

Files Changed	Coverage Δ
packages/seeder/src/SeedManager.ts	100.00% <ø> (ø)
packages/core/src/entity/ArrayCollection.ts	100.00% <100.00%> (ø)
packages/core/src/entity/Collection.ts	100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rubiin]

I was thinking instead of adding some minor collection methods in next pr, I will add them in here itself. What do you say

[B4nan]

i'd say it should be separate PRs so it results in multiple items in the changelog

and please pay more attention to the comments, you still havent addressed all the missing spaces and wrong indents

[rubiin]

I do have editorconfig extension setup and it should take care of the indentation afaik

[B4nan]

all right, last few style changes. sorry for not having the linter set up properly, I hate to waste time (yours as well as mine) discussing style changes like this.

thanks!

[rubiin]

all right, last few style changes. sorry for not having the linter set up properly, I hate to waste time (yours as well as mine) discussing style changes like this.

thanks!

No worries man. With codebase of this size, these are bound to happen.

[rubiin]

Btw about the collections test, should I just add tests for a specific driver under features/Collection.ts or will the file hold for all drivers

[B4nan]

We don't have to test this on all the drivers, in-memory sqlite is enough.