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

🎨 Rework prettier for code formatting #1950

Merged
merged 4 commits into from Apr 7, 2021
Merged

Conversation

cloud-walker
Copy link
Collaborator

@cloud-walker cloud-walker commented Apr 7, 2021

Contents

  • remove (unused) old prettier integration
  • add new prettier integration to the root of the project
  • add the yarn format command to enable auto formatting of the whitelisted files of the project

Supported files

The yarn format command currently support all the default extensions supported by prettier, so: .js, .ts, .json, .yml, .xml, .html, .md, etc

Todos

  • Understand the impact o the deployment process (maybe old prettier was used by some deployment script?)
  • Document the new yarn format command
  • Already expose the script to more files?

@cloud-walker cloud-walker self-assigned this Apr 7, 2021
@cloud-walker cloud-walker marked this pull request as ready for review April 7, 2021 15:13
@cloud-walker cloud-walker merged commit b70399d into develop Apr 7, 2021
@cloud-walker cloud-walker deleted the rework-prettier branch April 7, 2021 15:32
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.

None yet

2 participants