YouTube embed: use more privacy-friendly youtube-nocookie.com instead of youtube.com

[AlessioGr]

Europeans will be happy!

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
lexical	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 7, 2023 9:43pm
lexical-playground	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 7, 2023 9:43pm

[zurfyx]

LGTM, mind looking into the failing checks? Not too sure why it's failing, might be a bad rebase or something

[AlessioGr]

LGTM, mind looking into the failing checks? Not too sure why it's failing, might be a bad rebase or something

should hopefully be good now. One test was checking for youtube.com/embed instead of youtube-nocookie.com/embed. As for this:

This seems to potentially be a bug with the npm run ci-check command itself. At least I cannot explain it otherwise. This is how the command is defined:

"ci-check": "npm-run-all --parallel tsc flow prettier lint",

When I run tsc, flow, prettier and lint one by one manually, everything passes successfully. Thus, this might be an issue with npm-run-all?

[AlessioGr]

Running "npm run prettier" manually - no issues!

[acywatson]

Running "npm run prettier" manually - no issues!

This is a tricky one - npm run prettier actually runs with the flag --list-different, I think. The output is confusing.

Try npm run prettier:fix and then git status to see if anything changed.

[AlessioGr]

Running "npm run prettier" manually - no issues!

This is a tricky one - npm run prettier actually runs with the flag --list-different, I think. The output is confusing.

Try npm run prettier:fix and then git status to see if anything changed.

Aah, I see, found the issue! Yeah, that output is confusing indeed. Should be good now!

Do you by any chance know why VS Code doesn't automatically set prettier as the default formatter when I open the project? If I set it to prettier manually, it wants to do this to the settings.json:

[acywatson]

Running "npm run prettier" manually - no issues!

This is a tricky one - npm run prettier actually runs with the flag --list-different, I think. The output is confusing.
Try npm run prettier:fix and then git status to see if anything changed.

Aah, I see, found the issue! Yeah, that output is confusing indeed. Should be good now!

Do you by any chance know why VS Code doesn't automatically set prettier as the default formatter when I open the project? If I set it to prettier manually, it wants to do this to the settings.json:

I have no idea...you can probably commit that if it'd be helpful.