Skip to content
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

Upgrade prettier to v2.5.1 #19719

Merged

Conversation

JonathanCrd
Copy link
Member

Solves: #9329. Upgrade prettier dependencies to latest version (^2.5.1) in the following folders:

  • Root package.json
  • eng/tools/dependency-testing/
  • samples/
    • cors/ts/
    • frameworks/electron/
    • frameworks/electron/
  • sdk/
    • anomalydetector/
    • deviceupdate/
    • quantum/
    • videoanalyzer/

Files were re-formatted as well, with two exceptions:

  • In the samples/ folder, there was no format script, so only the version in package.json files was modified.
  • For sdk/quatum/quantum-jobs, the format script is skipped.

There are only format changes in this PR, no manual changes except for package.json files.

Main format changes with Prettier 2.x in this PR include:

  • Trailing commas by default.
  • Whitespace added after every function keyword.

@JonathanCrd JonathanCrd self-assigned this Jan 6, 2022
@@ -82,11 +82,11 @@
"build:samples": "echo Obsolete.",
"build:test": "echo skip",
"build": "npm run clean && tsc -p . && rollup -c 2>&1 && api-extractor run --local",
"check-format": "prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In order to fix the CI, it was necessary to remove the path "test/**/*.ts" from the format and check-format scripts.
This is due to iot-device-update not having a test folder, which throws an error when running any of those scripts.

@JonathanCrd JonathanCrd merged commit 69fa41b into Azure:main Jan 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants