[5.5] Make seeders chainable

[freekmurze]

Right now you must call seeders in DatabaseSeeder like this:

$this->call(SeederA::class);
$this->call(SeederB::class);
$this->call(SeederC::class);

The changes in this PR will enable to do the above like this

$this
   ->call(SeederA::class);
   ->call(SeederB::class);
   ->call(SeederC::class);

[antonkomarev]

What is the profit from it?

Wouldn't it be better to make:

$this->call([
    SeederA::class,
    SeederB::class,
    SeederC::class,
]);

Or:

$this->calls([
    SeederA::class,
    SeederB::class,
    SeederC::class,
]);

[devcircus]

Can't you already do

$this->call([
    SeederA::class,
    SeederB::class,
]);

[browner12]

@devcircus yes you can. I'm not really seeing the benefit of chaining, but oh well, it's already in.

[lbm-trentm]

if no one can really see the benefit to this, why bloat the software?

[nachofassini]

what @a-komarev proposes its already implemented. Seeder docs

[deleugpn]

@lbm-trentm The PR creator, people liking the PR and Taylor seem to disagree with your premise.

[antonkomarev]

@deleugpn nobody said that we are disagree with this PR. Question of reason was raised and this behavior was available already, as @nachofassini pointed to docs.

One minor notice that in first comment bad example is provided. It should be:

$this
   ->call(SeederA::class)
   ->call(SeederB::class)
   ->call(SeederC::class);

[connectkushal]

to those grumbling about bloating the software without any benefit, this pr adds just 2 extra words to the actual code, return $this 🙄

[browner12]

Lingchi - Death by 1000 cuts.

Mostly trolling, but our initial point was this was a solution to a problem that already had a solution.

It's in now, so /shrug