Coding Standards Meeting 2023-08-01 2100 UTC

Hello all, it’s time for the fortnightly coding standards meeting. The meeting will take place in slack in various threads
This meeting:
➤ Is for anyone interested in the Drupal coding standards.
➤ Usually happens fortnightly on Tuesday 2100 UTC. The meeting open for 24 hours to allow for all timezones.
➤ Is done on the #coding standards channel in Drupal Slack (see www.drupal.org/slack for information).
➤ Happens in threads, which you can follow to be notified of new replies even if you don’t comment in the thread. You may also join the meeting later and participate asynchronously!
➤ Has a public agenda anyone can add to. See the parent issue for an idea of the typical agenda.
➤A transcript will be made using drupal-meeting-parser and posted to the agenda issue. For anonymous comments, start with a :bust_in_silhouette: emoji. To take a comment or thread off the record, start with a :no_entry_sign: emoji.

0️⃣ Who is here today? Comment in the thread to introduce yourself. We’ll keep the meeting open for 24 hours to allow for all timezones.

1️⃣ Do you have suggested topics you are looking to discuss? Post in this thread and we’ll open threads for them as appropriate.

2️⃣ Action items

2️⃣.1️⃣ Approve previous minutes

2️⃣.2️⃣ Follow up work to do on #2707507: Always add trailing commas, for arrays and annotations (edited) 

4️⃣ Issues tagged "final discussion"

4️⃣.1️⃣ #3238192: Allow omitting @var for strictly typed class properties

4️⃣.2️⃣ #3324368: Update CSS coding standards to include PostCSS and Drupal 10

5️⃣ Issues tagged "Needs announcement for final discussion". These we need to decide if they are ready for a final announcment. (edited) 

5️⃣.1️⃣ #2464123: Remove the requirement that no blank line follow an inline comment (edited) 

    $this->clickLink('Cancel');
    // @todo Uncomment the following line after Core issue is fixed. See
    // #2582295: Confirmation cancel links are incorrect if installed in a subdirectory.
    // $this->assertSession()->addressEquals($overview_url);

    // Go to the next page.
    $this->drupalGet($next_page);

5️⃣.2️⃣ #3074131: Use null coalescing operator ?? instead of a ternary operator with an isset() condition

5️⃣.3️⃣ #2961791: Update Help Text Standards Examples to Modify Heading Structure

6️⃣ Last minutes change! #2140961: Allow constructor methods to omit docblocks

7️⃣  Activity in core

8️⃣  Share your opinions on this. This could be an easy fix. #3305090: Should core use cspell or cSpell

$ grep -rq cSpell core && echo failed
failed

9️⃣ Wrap up.  The meeting is open for 24 hours, Please continue to chat in the threads. A transcript is posted, usually within 1 week.