- Issue created by @bbrala
- Status changed to Needs review
over 1 year ago 9:19am 5 September 2023 - π³π±Netherlands bbrala Netherlands
- Status changed to Fixed
over 1 year ago 7:35am 23 September 2023 - π³π±Netherlands bbrala Netherlands
@agentrickard can you add credits and mark fixed please?
- Status changed to RTBC
over 1 year ago 7:35am 23 September 2023 - π¨π¦Canada paulsheldrake
The latest tag of drupal-rector is missing the D10 set https://github.com/palantirnet/drupal-rector/tree/0.15.1/src/Set. even though it is in the primary branch
- π³π±Netherlands bbrala Netherlands
Yeah its currently only released in main, we have not tagged a release. I think we will soon, but before that i want to have merged BC fixes into Rector.
β¨ Add backward compatibility to Rector rules Needs review
- π¨π¦Canada earthangelconsulting
hi there. adding myself to this thread so that i get notification when this is fixed. needless to say, checking for D10 compatibility is a big need for many of us right now ;-)
i had the same problem as @paulsheldrake (the D10 set missing) but at the time i was actually trying to test code for D9.5 compatibility, not D10, so I just removed the references to D10 from the rector.php configuration file, and used the more granular rule sets:
$rectorConfig->sets([ Drupal9SetList::DRUPAL_90, Drupal9SetList::DRUPAL_91, Drupal9SetList::DRUPAL_92, Drupal9SetList::DRUPAL_93, ]);
now i get a different error:
[ERROR] You have requested a non-existent parameter "drupal_rector_notices_as_comments"
is this related to the current issue, or the backward compatibility fixes at https://www.drupal.org/project/rector/issues/3350886 β¨ Add backward compatibility to Rector rules Needs review ?
if not i will start a new issue
- π¨π¦Canada earthangelconsulting
I am confused about how there could be all these upgrades to various modules, inserted into issues queues with subject "Automated Drupal 10 compatibility fixes", and mention as being as done by Drupal Rector... and yet there currently isn't a version of https://github.com/palantirnet/drupal-rector that works with D10? can anyone explain this?
- π³π±Netherlands bbrala Netherlands
This issue is mostly for checking for Drupal 11 compatibility. This would be deprecations that are added in the Drupal 10 lifecycle that need changing for Drupal 11.
- Status changed to Fixed
over 1 year ago 5:12pm 8 November 2023 - πΊπΈUnited States agentrickard Georgia (US)
This has been committed for the next release.
- π¨π¦Canada earthangelconsulting
re: #11 that's great to hear!
that error i was getting (see #8)
[ERROR] You have requested a non-existent parameter "drupal_rector_notices_as_comments"
i already found the fix, by looking at the rector-config-template.php file provided by the upgrade_rector module.
rector.php should contain this line:
$parameters->set('drupal_rector_notices_as_comments', true);
i hope this can make it into the next release too :-)
- πΊπΈUnited States agentrickard Georgia (US)
Yes. It's going in the release notes.
Automatically closed - issue fixed for 2 weeks with no activity.