- Issue created by @wim leers
- 🇬🇧United Kingdom ChrisDarke London
Migrating Pittsburgh 2023 to Pittsburgh2023 tag for cleanup
- First commit to issue fork.
- Assigned to nickdickinsonwilde
- last update
over 1 year ago 29,378 pass, 31 fail - @nickdickinsonwilde opened merge request.
- last update
over 1 year ago 29,378 pass, 31 fail - last update
over 1 year ago 24,524 pass, 31 fail - last update
over 1 year ago 24,563 pass, 13 fail - last update
over 1 year ago Custom Commands Failed - last update
over 1 year ago 24,576 pass, 5 fail - last update
over 1 year ago 24,581 pass, 3 fail - last update
over 1 year ago 29,437 pass, 1 fail - Issue was unassigned.
- 🇨🇦Canada nickdickinsonwilde Victoria, BC (T'So-uke lands)
All test errors fixed. However, that has exposed some missing config schema in aggregator module. I believe updating that is out of the scope of this ticket but needs to be resolved before this ticket can land.
- Assigned to nickdickinsonwilde
- 🇨🇦Canada nickdickinsonwilde Victoria, BC (T'So-uke lands)
oh doh! Of course aggregator doesn't have config. Because Aggregator module was deprecated in core in 9.4 and removed in 10.0.
Will examine that migration test and see what options there are to work around that. - Issue was unassigned.
- 🇨🇦Canada nickdickinsonwilde Victoria, BC (T'So-uke lands)
CR re aggregator: https://www.drupal.org/node/3267461 →
I think the best solution is to remove any tests related to migrating the aggregator module from core. but that might be beyond the scope of this issue still. So waiting for feedback/other ideas - Status changed to Needs work
over 1 year ago 7:05am 27 July 2023 - Status changed to Needs review
over 1 year ago 8:59am 27 July 2023 - last update
over 1 year ago 29,883 pass, 1 fail - Status changed to RTBC
over 1 year ago 9:03am 27 July 2023 - 🇧🇪Belgium borisson_ Mechelen, 🇧🇪
This looks great, probably should be mentioned in the CR for 📌 Fix all ConfigExists constraint violations in tests Fixed as well.
- Status changed to Needs work
over 1 year ago 1:10pm 27 July 2023 - Status changed to Needs review
over 1 year ago 1:17pm 27 July 2023 - Open on Drupal.org →Environment: PHP 8.2 & MySQL 8last update
over 1 year ago Not currently mergeable. - Status changed to Needs work
over 1 year ago 2:22pm 27 July 2023 - Status changed to Needs review
over 1 year ago 4:30pm 27 July 2023 - 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
This conflicted with 📌 `type: mapping` should use `ValidKeys: ` constraint by default Fixed ! Should be good again now :)
- last update
over 1 year ago 29,885 pass - Status changed to RTBC
over 1 year ago 6:27am 28 July 2023 - Status changed to Needs work
over 1 year ago 5:48pm 28 July 2023 - Status changed to Needs review
over 1 year ago 1:19pm 31 July 2023 - last update
over 1 year ago 29,911 pass - last update
over 1 year ago 29,912 pass - 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
📌 Fix all type: machine_name constraint violations in tests Fixed landed but didn't conflict with this. Still, merged in upstream changes to ensure accurate test results 👍
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Yay, 📌 Fix all PluginExistsConstraint constraint violations in tests Fixed and 📌 Add validation constraint to `type: label`: disallow multiple lines Fixed just landed — rebased! 👍
- last update
over 1 year ago 29,946 pass - Status changed to RTBC
over 1 year ago 4:01pm 1 August 2023 - 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
This was RTBC'd in #18. It was only un-RTBC'd for A) a reroll, B) adding docs. https://git.drupalcode.org/project/drupal/-/merge_requests/4138/diffs?co... added docs, no functional changes.
So: restoring RTBC status.
- Status changed to Fixed
over 1 year ago 5:09pm 1 August 2023 Automatically closed - issue fixed for 2 weeks with no activity.