- Issue created by @catch
- 🇺🇸United States smustgrave
haven't manually test but all tests appear to be failing.
- 🇺🇸United States smustgrave
Following the steps in the issue summary confirmed everything is behaving as normally.
The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- First commit to issue fork.
- 🇺🇸United States dcam
It didn't really need a rebase. Not sure why it thought the MR didn't apply.
- 🇫🇷France nod_ Lille
can't apply as a patch or rebase the MR from the UI, need some manual rebase/merge
- 🇺🇸United States dcam
@catch Sorry I hadn't finished the rebase yet. I was busy with work.
- 🇬🇧United Kingdom catch
Ahh apologies from me too, I missed that you were already doing the rebase (although I think I ended up rebasing on top of your in-progress rebase).
These are one of few MRs I don't mind rebasing because it's nearly always because the numbers are going down :)