-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update dependency prettier to v3 #1107
Conversation
Removed dependencies detected. Learn more about Socket for GitHub ↗︎ 🚮 Removed packages: prettier@2.8.8 |
ad579d0
to
6c3d8fa
Compare
f1ff751
to
c83d8a2
Compare
@mrose17 I see this pre-commit error due to this update:
|
OK, let's wait a week to see if "it fixes itself", e.g., in the same fashion as #1064 |
6d1f474
to
70fe22d
Compare
b6de628
to
f7ef8e9
Compare
b15d1e3
to
6a4eb9d
Compare
0c80e11
to
e4ecb0a
Compare
84a9daf
to
4a9a904
Compare
4a9a904
to
12de88b
Compare
12de88b
to
b7de7cf
Compare
- v3 makes 'trailingComma' default to true (see [release notes](https://prettier.io/blog/2023/07/05/3.0.0.html#change-the-default-value-for-trailingcomma-to-all-11479httpsgithubcomprettierprettierpull11479-by-fiskerhttpsgithubcomfisker-13143httpsgithubcomprettierprettierpull13143-by-sosukesuzukihttpsgithubcomsosukesuzuki)) - v3 can also cause failures with it's automatic jest integration, just disable this in jest.config.js since we reformat on commit anyway
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. ⚠ Warning: custom changes will be lost. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great,t hanks!
* Update all non-major dependencies (#1195) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update WelcomeScreen.tsx (#1205) Remove check for Brave browser when creating a room. * Update dependency node to v18.17.1 (#1208) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update dependency i18next to v23.4.2 (#1201) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1214) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update dependency eslint-config-prettier to v9 (#1210) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1217) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update use-params.ts (#1219) Change to make `?sol=y` as the default. In a few weeks, we'll remove the query flag altogether. * fix: correct prod location for labels-english (#1221) as noted by @kdenhartog, this file should be sourced from talk.brave.com * Updated: Implement web3 address exclusion behind a feature flag (#1215) * address exclusion * address exclusion * address exclusion * address exclusion * minor touch-ups 1. restore EOL on last line of modified files. 2. use simplified Unicode \s in regex * fix allow/deny address examples, update comments on query parameters * Update ExceptionListPanel.tsx courtesy of `npm run lint` * Sync with `main` package*.json * Update use-params.ts Fix conflict from #1219 --------- Co-authored-by: Marshall T. Rose <mrose17@gmail.com> * Update all non-major dependencies (#1225) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore: repository housekeeping (#1237) 1. Add an `npm run check` command that runs all of the commonly failing checks: eslint, tsc, prettier and audit. 2. Run `check` as part of the CI build 3. Ensure linter and prettier run on the pre-commit hook 4. Fix the linter and formatting errors that had crept in while these checks were not enforced * Update all non-major dependencies (#1227) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update dependency prettier to v3 (#1107) * Update dependency prettier to v3 * Update formatting based on prettier v3 - v3 makes 'trailingComma' default to true (see [release notes](https://prettier.io/blog/2023/07/05/3.0.0.html#change-the-default-value-for-trailingcomma-to-all-11479httpsgithubcomprettierprettierpull11479-by-fiskerhttpsgithubcomfisker-13143httpsgithubcomprettierprettierpull13143-by-sosukesuzukihttpsgithubcomsosukesuzuki)) - v3 can also cause failures with it's automatic jest integration, just disable this in jest.config.js since we reformat on commit anyway --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Graham Tackley <gtackley@brave.com> * chore: configure eslint explicitly (#1243) We used to rely on `eslint-config-react-app` to configure some aspects of eslint. This is the full-service config used by "Create React App". However it brings in other dependencies such as (an old version of) eslint itself. It also is much less maintained than it used to be, as many projects (including talk!) have moved away from using create-react-app. Use the react specific plugins directly. As part of this, eslint-plugin-react doesn't really like the style of declaring react functional components using `React.FC`. This is a bit of an outdated style anyway, and it's usually clearer just to create functions with explicit types. Since this is also how most react documentation sites do it, I've removed all uses of the `React.FC` type. Co-authored-by: Marshall T. Rose <mrose17@gmail.com> * Update typescript-eslint monorepo to v6 (#1136) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1242) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update aws-actions/configure-aws-credentials action to v4 (#1234) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update actions/checkout action to v4 (#1229) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update dependency node to v18.18.0 (#1249) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Marshall T. Rose <mrose17@gmail.com> * Update all non-major dependencies (#1250) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * renovate.json: add org-level base (#1253) * set `roomPasswordNumberOfDigits` to false to allow for alpanumeric passcodes thank you @damencho * Update all non-major dependencies (#1251) (#1256) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Graham Tackley <gtackley@brave.com> Co-authored-by: Hadi Amjad <46374292+hadiamjad@users.noreply.github.com> Co-authored-by: Andrea Brancaleoni <miwaxe@gmail.com>
This PR contains the following updates:
2.8.8
->3.0.3
Release Notes
prettier/prettier (prettier)
v3.0.3
Compare Source
diff
Add
preferUnplugged: true
topackage.json
(#15169 by @fisker and @so1ve)Prettier v3 uses dynamic imports, user will need to unplug Prettier when Yarn's PnP mode is enabled, add
preferUnplugged: true
topackage.json
, so Yarn will install Prettier as unplug by default.Support shared config that forbids
require()
(#15233 by @fisker)If an external shared config package is used, and the package
exports
don't haverequire
ordefault
export.In Prettier 3.0.2 Prettier fails when attempt to
require()
the package, and throws an error.Allow argument of
require()
to break (#15256 by @fisker)Do not print trailing commas in arrow function type parameter lists in
ts
code blocks (#15286 by @sosukesuzuki)Support TypeScript 5.2
using
/await using
declaration (#15321 by @sosukesuzuki)Support for the upcoming Explicit Resource Management feature in ECMAScript.
using
/await using
declarationv3.0.2
Compare Source
diff
Break after
=
of assignment if RHS is poorly breakable AwaitExpression or YieldExpression (#15204 by @seiyab)Do not add trailing comma for grouped scss comments (#15217 by @auvred)
Print
declare
andexport
keywords for nested namespace (#15249 by @sosukesuzuki)v3.0.1
Compare Source
diff
Fix cursor positioning for a special case (#14812 by @fisker)
Fix plugins/estree.d.ts to make it a module (#15018 by @kingyue737)
Add
export {}
inplugins/estree.d.ts
to fix the "File is not a module" errorAdd parenthesis around leading multiline comment in return statement (#15037 by @auvred)
Add support for Vue "Generic Components" (#15066 by @auvred)
https://blog.vuejs.org/posts/vue-3-3#generic-components
Fix comments print in
IfStatement
(#15076 by @fisker)Add missing type definition for
printer.preprocess
(#15123 by @so1ve)Add missing
getVisitorKeys
method type definition forPrinter
(#15125 by @auvred)Add typing to support
readonly
array properties of AST Node (#15127 by @auvred)Add space before unary minus followed by a function call (#15129 by @pamelalozano)
v3.0.0
Compare Source
diff
🔗 Release Notes
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.