Commit Graph

207 Commits

Author SHA1 Message Date
Nicolas Grekas
b1fbd1b8f1 bug #42099 [VarDumper] Support for intersection types (derrabus)
This PR was merged into the 4.4 branch.

Discussion
----------

[VarDumper] Support for intersection types

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | #41552
| License       | MIT
| Doc PR        | N/A

Commits
-------

3cf105766a [VarDumper] Support for intersection types
2021-07-15 09:45:25 +02:00
Tobias Schultze
e5c96c4051 bug #42011 [Cache] Support decorated Dbal drivers in PdoAdapter (Jeroeny)
This PR was squashed before being merged into the 4.4 branch.

Discussion
----------

[Cache] Support decorated Dbal drivers in PdoAdapter

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Explanation in this PR
| License       | MIT

Doctrine v3 supports middleware for Drivers. Upon creating the Connection, `middleware->wrap(Driver): Driver` [is called](https://github.com/doctrine/dbal/blob/3.1.x/src/DriverManager.php#L210), in which the middleware can wrap/decorate the Driver class. So that it can perform tracing [for example](https://github.com/getsentry/sentry-symfony/blob/master/src/Tracing/Doctrine/DBAL/TracingDriverMiddleware.php#L37).

When this happens, the Driver class inside the Connection is no longer one of Doctrine's well known Driver classes. The `PdoAdapter ` uses this class to determine the database platform. Which breaks once the Driver is decorated and no longer one of the classes [listed](https://github.com/symfony/symfony/blob/5.4/src/Symfony/Component/Cache/Adapter/PdoAdapter.php#L452) in the `PdoAdapter`.

Since Dbal exposes this middleware as a feature, I think it would be nice for the `PdoAdapter` to support this.

To solve this, the `getDatabasePlatform` can be used. This returns a `Doctrine\DBAL\Platforms\AbstractPlatform` which defines the abstract method `getName`. This returns a value very similar to the list in the `PdoAdapter`. The names don't match exactly, so therefor a small mapping is done to get right the name used in the adapter. As far as a I know, there'd be no other implications with this change.

Related: https://github.com/getsentry/sentry-symfony/issues/530

Commits
-------

58d74e30be [Cache] Support decorated Dbal drivers in PdoAdapter
2021-07-14 22:18:05 +02:00
Jeroeny
58d74e30be [Cache] Support decorated Dbal drivers in PdoAdapter 2021-07-14 22:17:56 +02:00
Alexander M. Turek
3cf105766a [VarDumper] Support for intersection types
Signed-off-by: Alexander M. Turek <me@derrabus.de>
2021-07-14 16:06:50 +02:00
Nicolas Grekas
becf11aa14 Fix tests 2021-07-13 11:55:30 +02:00
Fabien Potencier
09814d5548 minor #42007 [GHA] Clarify some bits in the deps=high script (wouterj)
This PR was merged into the 4.4 branch.

Discussion
----------

[GHA] Clarify some bits in the deps=high script

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix -
| License       | MIT
| Doc PR        |

I had some difficulty understanding how flipped tests worked. Luckily, `@nicolas`-grekas was available to answer some questions. However, I'm sure that I've forgotten the logic in about a week time, so I think it's worth clarifying the code a bit (but feel free to close if it's just me).

Commits
-------

ff08dcaca6 [GHA] Clarify some bits in the deps=high script
2021-07-08 11:16:39 +02:00
Nicolas Grekas
a6a00915c3 [Cache] backport type fixes 2021-07-07 14:25:09 +02:00
Wouter de Jong
ff08dcaca6 [GHA] Clarify some bits in the deps=high script 2021-07-06 18:29:48 +02:00
Alexander M. Turek
44ca13ef40 Revert CI workaround for masterminds/html5
Signed-off-by: Alexander M. Turek <me@derrabus.de>
2021-07-01 16:52:12 +02:00
Nicolas Grekas
7c6c7b0331 Fix tests 2021-06-30 16:07:32 +02:00
Nicolas Grekas
edf74fa9ab Tweak GHA 2021-06-29 11:23:13 +02:00
Nicolas Grekas
21621ab783 [FrameworkBundle] fix tests 2021-06-28 17:39:02 +02:00
Nicolas Grekas
ac2c3a936d Tweak CI 2021-06-28 11:04:43 +02:00
Nicolas Grekas
ab2a0fe653 Tweak CI 2021-06-28 11:03:12 +02:00
Nicolas Grekas
bd412f2616 Tweak GHA 2021-06-27 18:37:09 +02:00
Nicolas Grekas
058168e2ed [gha] Define COLUMNS properly
This reverts commit c2e4ac613c.
2021-06-27 14:18:17 +02:00
Nicolas Grekas
0ef5ec8f81 - 2021-06-27 14:04:45 +02:00
Nicolas Grekas
c2e4ac613c - 2021-06-27 13:58:40 +02:00
Nicolas Grekas
bb59651e48 Fix CI 2021-06-27 13:30:21 +02:00
Nicolas Grekas
fa5ed19ab8 - 2021-06-26 23:56:04 +02:00
Alexander M. Turek
b8182a8ecc PHPUnit on GitHub Actions 2021-06-26 13:58:58 +02:00
Alexander M. Turek
9c69e77805 Pin masterminds/html5 to the master branch for PHP 8.1
Signed-off-by: Alexander M. Turek <me@derrabus.de>
2021-06-22 17:42:29 +02:00
Nicolas Grekas
a71fb4b8f2 fix tests (ter) 2021-06-07 16:39:01 +02:00
Fabien Potencier
36238a4134 minor #41281 Add 5.3 in github PR template (jderusse)
This PR was merged into the 4.4 branch.

Discussion
----------

Add 5.3 in github PR template

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

Right time now :)

Commits
-------

327014b104 Add 5.3 in github PR template
2021-05-19 15:46:28 +02:00
Jérémy Derussé
327014b104
Add 5.3 in github PR template 2021-05-19 14:57:42 +02:00
Tobias Nyholm
6a328ae01e
Update .github/workflows/tests.yml 2021-05-15 10:42:08 +02:00
Alexander M. Turek
6f1c4b8c58 PHP 8.1 must not cause the build to fail (yet) 2021-05-14 20:01:46 +02:00
Alexander M. Turek
b0b45105d7 Add _run_tests bash function 2021-05-14 19:21:16 +02:00
Alexander M. Turek
de4f0170e2 Add a GitHub Action for tests against PHP 8.1 2021-05-14 17:37:17 +02:00
Nyholm
2345614316 [CI][Psalm] Install stable/released PHPUnit 2021-05-12 18:07:54 +02:00
Nyholm
d36f5e31ee
[CI][Psalm] Install stable/released PHPUnit 2021-05-12 08:32:34 +02:00
Nicolas Grekas
9ba015d236 [travis] remove cache of composer.lock for deps=low 2021-03-23 21:57:14 +01:00
Nicolas Grekas
af43ca080b [travis] use packagist API v2 2021-03-23 20:31:36 +01:00
Nyholm
3c47e03e92
Specify that we run CI on Ubuntu-20.04 2021-03-02 10:27:35 +01:00
Nicolas Grekas
3fb74abe62 Speedup psalm 2021-02-26 00:35:49 +01:00
Nyholm
d5a05f1b30 Make sure the Psalm review CI job is working 2021-02-25 17:51:23 +01:00
Nyholm
c5ed24d8cb Adding a Github action to run Psalm 2021-02-25 17:18:18 +01:00
Nicolas Grekas
56194a4535 Move github templates at the org level 2021-02-23 11:03:30 +01:00
Robin Chalas
c28ba9a959 Remove EOLed 5.1 branch from PR template 2021-02-05 15:36:10 +01:00
Antoine Makdessi
41c7796552 Update PULL_REQUEST_TEMPLATE.md 2021-02-05 09:45:51 +01:00
Nicolas Grekas
b40e71f096 Replace "branch-version" by "versions" in composer.json 2021-01-27 17:22:20 +01:00
Nicolas Grekas
7f5ea78fb8 [travis] use PHP 8.0 to patch return types and run deps=low 2021-01-14 15:31:07 +01:00
Jérémy Derussé
9a3fc5a967
Fix bitnami env PATH 2021-01-07 00:54:10 +01:00
Nicolas Grekas
09e51d6b42 Cleanup CI scripts 2021-01-04 17:51:14 +01:00
Alexander M. Turek
0bc15ff759 [PhpUnitBridge] Lint files with PHP 5.5 2020-12-14 16:00:48 +01:00
Jakub Zalas
745020892f
Run intl-data tests on resources change 2020-12-07 20:17:10 +00:00
Fabien Potencier
bda2dcdf93 bug #39251 [DependencyInjection] Fix container linter for union types (derrabus)
This PR was merged into the 4.4 branch.

Discussion
----------

[DependencyInjection] Fix container linter for union types

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #39233
| License       | MIT
| Doc PR        | N/A

Commits
-------

e26893b122 [DependencyInjection] Fix container linter for union types.
2020-12-05 18:12:03 +01:00
Jan Schädlich
c53a6d04c8 Fix github pr template and include 5.2 for bugfixes 2020-12-04 22:21:05 +01:00
Alexander M. Turek
e26893b122 [DependencyInjection] Fix container linter for union types. 2020-11-30 22:45:46 +01:00
Fabien Potencier
728574d94f minor #39095 [Cache] Run Redis Sentinel tests in GithubAction (jderusse)
This PR was merged into the 4.4 branch.

Discussion
----------

[Cache] Run Redis Sentinel tests in GithubAction

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

Add support for Sentinel tests in GHA

Commits
-------

bd84394dc5 Run Redis Sentinel tests in GithubAction
2020-11-17 06:55:49 +01:00