Update composer OR in constraints to use newer double pipe

[jessiegreen]

Please take note of our contributing guidelines: https://docs.laravel-excel.com/3.1/getting-started/contributing.html
Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.

1️⃣ Why should it be added? What are the benefits of this change?
I was unable to composer require maatwebsite/excel:^3.1 with Composer version 2.6.1 and PHP 8.2.10 getting "maatwebsite/excel 3.1.0 requires php ^7.0 -> your php version (8.2.10) does not satisfy that requirement" with the single pipe.

2️⃣ Does it contain multiple, unrelated changes? Please separate the PRs out.
No

3️⃣ Does it include tests, if possible?
N/A

4️⃣ Any drawbacks? Possible breaking changes?
This is the latest and should not break anything

5️⃣ Mark the following tasks as done:

• [x ] Checked the codebase to ensure that your feature doesn't already exist.
• [x ] Take note of the contributing guidelines.
• [x ] Checked the pull requests to ensure that another person hasn't already submitted a fix.
• [x ] Added tests to ensure against regression.

6️⃣ Thanks for contributing! 🙌

[patrickbrouwers]

Are you sure that has to do with the pipes and not with the fact that you might be missing a php ext that is required by newer php spreadsheet versions? Do you know the difference beteeen | and || ?

[jessiegreen]

I was able to fork the project, only update the pipes, and then it worked with no issue. Single pipe is deprecated for composer. It was supposed to have backwards compatibility but there seems to be a clear behavior difference in the most recent version. https://getcomposer.org/doc/articles/versions.md#version-range only mentions double pipe.

…

On Tue, Oct 31, 2023 at 1:09 PM Patrick Brouwers ***@***.***> wrote: Are you sure that has to do with the pipes and not with the fact that you might be missing a php ext that is required by newer php spreadsheet versions? Do you know the difference beteeen | and || ? — Reply to this email directly, view it on GitHub <#4022 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAHRQLUUMPIKYQVHTW7SLWDYCE5GBAVCNFSM6AAAAAA6X24ZXSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBXG4ZTCNRSGI> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[patrickbrouwers]

Awesome, sounds like this will solve a major pain. Thanks for testing it and linking the relevant documentation!