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

Format VSCode extension code with Prettier #1178

Merged

Conversation

dotNomad
Copy link
Collaborator

@dotNomad dotNomad commented Mar 20, 2024

Sets up Prettier to format our VSCode extension. #1180 contains the actual formatting changes which will be merged into this to get CI passing - keeping it separate for now for easy review.

Intent

Part of #1030

Type of Change

    • Bug Fix
    • New Feature
    • Breaking Change
    • Documentation
    • Refactor
    • Tooling

Approach

  • Uses eslint-config-prettier to ensure that Prettier and ESLint play nicely together
  • Uses an exact install of Prettier as recommended to avoid formatting changes on patch versions
  • Adds just commands and npm scripts
  • Adds just vscode check-format in the same GitHub workflows that run the linting

@dotNomad dotNomad force-pushed the dotnomad/vscode-prettier branch from 1971121 to 04d027d Compare March 20, 2024 20:29
@dotNomad dotNomad force-pushed the dotnomad/vscode-prettier branch 2 times, most recently from fcd3533 to c445189 Compare March 20, 2024 20:46
@dotNomad dotNomad marked this pull request as ready for review March 20, 2024 21:12
Copy link
Collaborator

@sagerb sagerb left a comment

Choose a reason for hiding this comment

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

LGTM

@sagerb sagerb changed the base branch from main to prettier-introduction-integration-branch March 21, 2024 18:06
@sagerb sagerb merged commit b1848ee into prettier-introduction-integration-branch Mar 21, 2024
34 of 42 checks passed
@sagerb sagerb deleted the dotnomad/vscode-prettier branch March 21, 2024 18:08
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.

2 participants