Skip to content

Chore: Format codebase with Prettier #663

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

Merged
merged 1 commit into from
Feb 28, 2025

Conversation

MaoShizhong
Copy link
Contributor

Because

Since adding .prettierrc, this codebase has long been in need of a global formatting so everything's on the same page. As per #658 (review), that time is now.

This PR

  • Runs a single pass of npm run format -- . (no other changes made)

Issue

N/A

Additional Information

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project Contributing Guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Callbacks command: Update verbiage
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If this PR adds new features or functionality, I have added new tests
  • If applicable, I have ensured all tests related to any command files included in this PR pass, and/or all snapshots are up to date

Single pass of `npm run format -- .` without additional changes.
@MaoShizhong
Copy link
Contributor Author

Purposely left out .circleci/ in case that clashes with #648. It's being deleted in that PR so no need to format it.

@JustWaveThings JustWaveThings self-requested a review February 28, 2025 02:02
Copy link
Contributor

@JustWaveThings JustWaveThings left a comment

Choose a reason for hiding this comment

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

Thanks Mao!

@JustWaveThings JustWaveThings merged commit 7b8faeb into TheOdinProject:main Feb 28, 2025
3 checks passed
@MaoShizhong MaoShizhong deleted the format-prettier branch March 4, 2025 16:01
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