Skip to content

Apply formatting rules#103

Merged
fexpal merged 8 commits intoeclipse-editdor:masterfrom
egekorkan:fix-formatting
Mar 17, 2025
Merged

Apply formatting rules#103
fexpal merged 8 commits intoeclipse-editdor:masterfrom
egekorkan:fix-formatting

Conversation

@egekorkan
Copy link
Contributor

We have noticed with @TejInaco that the prettierrc was actually never applied and there were already some discrepencies in the codebase such as:

  • mixed usage of tabs and spaces
  • different indentation

Running prettier was not possible as well as the default parser was babel. I have adjusted the prettier configuration to reduce the diff as well

@egekorkan
Copy link
Contributor Author

@fexpal can you have a look and merge this PR? The only changes are in the package.json (adding devDependency), prettierrc and prettierignore. We can also change the prettier configuration to use other indentations etc.

@fexpal fexpal self-assigned this Mar 17, 2025
@fexpal fexpal merged commit 7cf0a1f into eclipse-editdor:master Mar 17, 2025
1 check passed
guneet-baweja pushed a commit to guneet-baweja/editdor that referenced this pull request Jan 10, 2026
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