+
Skip to content

Conversation

mmethner
Copy link

@mmethner mmethner commented Sep 5, 2019

This updates the Readme to explain conditional queries and how \Spot\Query::addWhereOperator
Thanks to @marcojetson for his work on #161 back in 2016.

@mmethner
Copy link
Author

Hi @vlucas I would like to add a PR for php8 support, but this pr seems to be merged first, before I contribute the next pr. Would you like to review and merge?

"vlucas/valitron": "~1.1",
"doctrine/dbal": "^2.5.4",
"sabre/event": "~2.0"
"php": ">= 8.1.11",
Copy link
Collaborator

Choose a reason for hiding this comment

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

If the PHP version is going to be bumped up to PHP8+, then we will have to do a major version release since this is a huge breaking change.

Copy link
Author

Choose a reason for hiding this comment

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

Hi @vlucas It looks like I did something wrong with the branch and repository I pushed to. Especially the last commit today was addressed to my fork. The compatiblity for 8.1 nevertheless is given. Are you willing to make a release out of it?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes, I can do a major release to support PHP8+.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载