Skip to content

Commit

Permalink
drop PHP version requirement, as it is enforced by illuminate/databas…
Browse files Browse the repository at this point in the history
…e package
  • Loading branch information
yurii-github committed Nov 15, 2020
1 parent 1997a86 commit 28ebcc0
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@
}
],
"require": {
"php": ">=7.1",
"illuminate/database": ">=5.6 <9.0"
},
"require-dev": {
Expand Down

4 comments on commit 28ebcc0

@vivekDhyani01
Copy link

@vivekDhyani01 vivekDhyani01 commented on 28ebcc0 Jul 10, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi i'm using php verson :- 7.0.33
and i'm stuck with the illuminate/database dependency as it require upper version of php
please help with package valid for version 7.0

illuminate/database 5.7.17 requires php ^7.1.3 -> your PHP version (7.0.33) does not satisfy that requirement.

@yurii-github
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hello. where did you find this PHP dependency? I remember removing PHP dependency in favor of Illuminate database dependency only.

@vivekDhyani01
Copy link

@vivekDhyani01 vivekDhyani01 commented on 28ebcc0 Jul 12, 2021 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yurii-github
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://github.com/illuminate/database/blob/v5.6.39/composer.json
has dependency on 7.1, so you can upgrade

About 7.0 support, I remember maintainer allowed to drop 7.0 support, so I did as it was adding a lot of workarounds.

Please sign in to comment.