Fix Drupal.Commenting.VariableComment.Missing in Unit tests

Restore some variables that should be removed in 📌 Remove 2 unused class properties in tests Active

Seems to be 1 open thread. that needs addressing (from you), maybe good for a first time user?

Was any of the other changes reviewed?

Were any of the other changes reviewed?

Yes

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.

I think the bot was wrong but I rebased anyway. There were no conflicts, so back to RTBC

Rebased and made changes for recently added code, so back to NR.

Rebase seems good

The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. 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.

Simple rebase, so restoring RTBC

The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. 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.

Rebase seems good. Way more patient then myself to keep up with all these :)

Needs another reroll sorry

The conflict was with the phpstan baseline so I rebuilt it. Linting passes, so I am restoring the RTBC

Committed to 11.x - thanks!

larowlan → closed merge request !9688

Automatically closed - issue fixed for 2 weeks with no activity.