Update Stylelint and Prettier for Drupal 10.1 and use Prettier for formatting PostCSS

Always for updating.

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. 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.

Moving back to RTBC

Assuming if updating the package broke it would of been caught by the build.

Committed/pushed to 10.1.x, thanks!

From what I can tell majority of css changes are just spacing updates.

Did notice .css files changed from inline to

+.ui-icon-contact {
+ background-position: -192px -128px;
+}

But since Drupal uses aggregation imagine that's a non issue.

Setting this back to NW based on the #15's code checks failing

From the output (https://dispatcher.drupalci.org/job/drupal_patches/177539/consoleText), I see

ERROR: The compiled CSS from the PCSS files
       does not match the current CSS files. Some added
       or updated JavaScript package made changes.
       Recompile the CSS with: yarn run build:css

There were some whitespace issues in core/themes/claro/css/components/progress.css file. I have provided the patch and interdiff for same.

This looks great!

I went through the code (including all of the CSS changes) and didn't see anything out-of-line. I also tested the script locally running both yarn build:css and yarn lint:css without any changes (or issues).

Needs a reroll, unsure why TestBot didn't kick this back to NR.

Weird error coming from the test bot:

Drupal's JavaScript development dependencies are not installed or cannot be resolved. Run 'yarn install' inside the core directory, or 'yarn check -s' to list other errors.

Needs a reroll after #3343212: Update Core CSS to use double-colon for pseudo elements landed.

This looks perfect (pending tests)!

I verified the only changes in the CSS are minor white-space changes (adding/removing lines, spacing inside of calc(), and spacing next to comments).

I also verified the yarn build, yarn build:css, yarn lint:css, and yarn watch:css commands work as expected, and don't generates results that differ from whats in the patch.

Adding tag for Midcamp, since I reviewed that while attending :)

This needs yet another re-roll for yarn.lock changes.

Fixed the conflict in the yarn.lock file. Not really sure how to generate a useful interdiff, though. I had to apply the changes before a prev commit, stash / apply them and fix the file.

Setting back to NR

Back to RTBC pending the test run.

error: patch failed: core/themes/claro/css/components/toolbar.module.css:202
error: core/themes/claro/css/components/toolbar.module.css: patch does not apply

And again... We might need to schedule this one and avoid other CSS patches around a commit window if this keeps happening.

Going to be bold and commit this while the iron's hot (i.e. while tests are still running).

Committed 53f1304 and pushed to 10.1.x. Thanks!

I published the change record.