Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge release 2.12.0 into 3.0.x #4379

Closed

Conversation

github-actions[bot]
Copy link

Release 2.12.0

2.12.0

  • Total issues resolved: 1
  • Total pull requests resolved: 7
  • Total contributors: 5

Documentation,Static Analysis

CI

Bug,Static Analysis

Documentation,Error Handling

CI,PHP

PHP

morozov and others added 25 commits October 18, 2020 19:01
It has support for PHP8.
That test will get remove in next major release, and requires too much
maintenance.
Jobs that use other versions of PHP are temporarily removed so that it's
easier to focus and iterate.
This leverages a new feature of shivammathur/setup-php that allows to
fail the build if an extension or tool fails to install.
Move the logic out of the assertion
My IDE was telling me I was trying to catch a non-existent exception, however it's clear by digging down a level that the Exception is throw by this class, so this simply annotates that.

Perhaps this should catch it and throw a QueryException?
Test the newest version, the oldest version, and versions for which we
have version-specific code.
This is based on platform classes that have a version number in their
name.
Psalm fails on release commits
Fail on extension / tool installation failure
…notation

Adds exception thrown by execute() method
Configuration should not be internal
@morozov morozov closed this Oct 22, 2020
@morozov morozov deleted the 2.12.x-merge-up-into-3.0.x_5f91cf8a00c230.67462317 branch October 22, 2020 18:52
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants