The Needs Review Queue Bot โ tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide โ to find step-by-step guides for working with issues.
- ๐ฎ๐ณIndia nikhil_110
Attached patch against Drupal 10.1.x
Patch #9 is not applied for Drupal 10.1.x so Inter-diff file is not added.
- last update
over 1 year ago 29,371 pass - Status changed to Needs review
5 months ago 12:01pm 31 July 2024 - Status changed to Needs work
5 months ago 12:09pm 31 July 2024 The Needs Review Queue Bot โ tested this issue.
While you are making the above changes, we recommend that you convert this patch to a merge request โ . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
- Status changed to Needs review
5 months ago 2:28pm 31 July 2024 - Status changed to RTBC
5 months ago 2:53pm 5 August 2024 - Status changed to Needs work
4 months ago 8:59am 20 August 2024 - ๐ซ๐ทFrance nod_ Lille
This doesn't seem enough, we need a variable name. In PHP you need to do
function test(...$params) {}
for this to work, so doc needs to be
@param string โฆ$params
, we can't have only theโฆ
. - Status changed to Needs review
4 months ago 10:17pm 20 August 2024 - ๐ณ๐ฟNew Zealand quietone
@nod_, thanks. I converted that without much though, sorry.
- Status changed to Needs work
4 months ago 10:20pm 20 August 2024