Skip to content

added prettier.io as an experiment for code formatting#190

Open
reusr1 wants to merge 3 commits into
develop-sling12from
code-standard-prettier
Open

added prettier.io as an experiment for code formatting#190
reusr1 wants to merge 3 commits into
develop-sling12from
code-standard-prettier

Conversation

@reusr1

@reusr1 reusr1 commented Apr 14, 2021

Copy link
Copy Markdown
Contributor
  • code style format through prettier*

as an alternative to #189 I tried to use prettier (using their defaults)

npx prettier --write .

observation: if you run percli hatch "*" we of course revert a lot of code back to a style that is not desired

@reusr1 reusr1 mentioned this pull request Apr 14, 2021
@cmrockwell

Copy link
Copy Markdown
Collaborator

This PR seems to have the outcome of standards applied committed along with the added config file .prettierrc.json
It would be nicer to commit the changes to the build and new files like .prettierrc.json separated from the work of applying the standards. Maybe something like https://github.com/HubSpot/prettier-maven-plugin ?

@reusr1

reusr1 commented Apr 28, 2021

Copy link
Copy Markdown
Contributor Author

This PR seems to have the outcome of standards applied committed along with the added config file .prettierrc.json
It would be nicer to commit the changes to the build and new files like .prettierrc.json separated from the work of applying the standards. Maybe something like https://github.com/HubSpot/prettier-maven-plugin ?

@cmrockwell the PR was mostly to illustrate the result of prettier.io and if that would be acceptable by others or if we want to use what @Felix-Puetz proposed in #189

@reggie7 reggie7 self-requested a review May 12, 2021 09:16
@reusr1

reusr1 commented May 12, 2021

Copy link
Copy Markdown
Contributor Author

looking at the conflicts and the output a bit more it seems we first have to fix the output from our code generator to work better with enabling prettier.io

also wondering if we should move to 2 spaces instead of 4 for the tabs

@flx-sta

flx-sta commented May 12, 2021

Copy link
Copy Markdown
Collaborator

Why don't we just start with something?`and keep on going from there.
It's just code-format..

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.

4 participants