[5.6] Tightenco/Collect should not be replaced anymore

[antonioribeiro]

It's now under on its own namespace to co-live with Illuminate\Collection. And this is preventing us to require it.

[mattstauffer]

👍 Thanks Antonio!

[crynobone]

1. Shouldn't we use "tighten/collect":"<5.6.0" since it still would conflict with versions before v5.6.0.
2. PR to 5.5 as well?

[ronjunevaldoz]

I love this fixed thanks! @antonioribeiro

[mattstauffer]

@crynobone Great question. I really wish we could backport the alias, because it's really not version specific. But you're right; older versions will still have this same issue.

And yah, I do think it should apply to multiple versions.

@antonioribeiro would you be up for making the requested changes?

[antonioribeiro]

@mattstauffer, already done:

#23152 - 5.5
#23153 - 5.6

@crynobone, thank you.

[antonioribeiro]

Note: the namespace change happened in tightenco/collect 5.5.33, that's why it's <5.5.33, after that there's no need to replace collect anymore.

[crynobone]

@antonioribeiro One more thing, https://github.com/laravel/framework/blob/5.6/src/Illuminate/Support/composer.json#L24

[antonioribeiro]

Done, @crynobone:

illuminate/support#61
illuminate/support#62

[antonioribeiro]

Those are the correct PRs:

#23159
#23160