- Issue created by @nickdickinsonwilde
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
liam morland β made their first commit to this issueβs fork.
- First commit to issue fork.
- π³π±Netherlands ekes
Reviewed and spotted one last nullable type. For the rest looks good.
Note this does not change the function signature see my note about this on https://github.com/commerceguys/addressing/pull/221#issuecomment-2582855457 so shouldn't require more than a patch release.
- π¨πSwitzerland berdir Switzerland
.gitlab-ci.yml changes apart from temporarily enabling PHP_MAX are out of scope. Those performance optimizations are very much on purpose. the phpunit jobs took 30min+ here.
As commented in the other issue, the job output is full of entity_browser and search_api (and maybe other) nullable types errors, which makes it hard to verify that ther's nothing left in paragraphs.
While not a hard blocker, it would be helpful to make sure issues in those projects exist and are referenced.
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
I have restored the testing customization and referred to the related issues in
entity_browser
andsearch_api
. - Status changed to RTBC
2 months ago 2:41pm 18 February 2025 - π¦πΉAustria klausi π¦πΉ Vienna
Looks good, thanks!
Attaching a stable patch file for composer patches.
- π¨πSwitzerland berdir Switzerland
Merged. field_group and search_api still cause some fails, removed max testing for now.
-
berdir β
committed 0c6e7260 on 8.x-1.x authored by
nickdickinsonwilde β
Issue #3492419 by liam morland, nickdickinsonwilde, ekes: Nullable types...
-
berdir β
committed 0c6e7260 on 8.x-1.x authored by
nickdickinsonwilde β
- π§πͺBelgium aspilicious
@berdir is it possible to roll a new release?
Automatically closed - issue fixed for 2 weeks with no activity.