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.

Sign up for GitHub

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

Jump to bottom

fix stylistic rules #1244

Draft
wants to merge 92 commits into
base: main
Choose a base branch
from
Draft

fix stylistic rules #1244

wants to merge 92 commits into from

Conversation

nevio18324
Copy link
Collaborator

No description provided.

kcinay055679 and others added 30 commits December 16, 2024 08:08
@kcinay055679
introduce spotless formatter
3f4c16e
@kcinay055679
update readme and pipeline
cfcef31
@kcinay055679
add typescript formatter
d4bbe7f
@kcinay055679
add frontedn config
57f8e2d
@kcinay055679
update tsconfig.json
d3fb549
@kcinay055679
install eslint for formating
7c543f1
@Miguel7373 @kcinay055679
update typescript-eslint to 8.15.0
194fac2
@Miguel7373 @kcinay055679
edit config
681b838
@Miguel7373 @kcinay055679
put unused-imports plugin into config
3872302
@Miguel7373 @kcinay055679
change config for spec files
a6d8322
@kcinay055679
add stylistic plugin to eslnt
83cb588
@kcinay055679
add eslint stylistic rules
c3b2f92
@kcinay055679
update config for eslint stylic
1ef9940
@kcinay055679
styling indent set to 2
ff7aff7
@kcinay055679
add html formatting
74fdf74
@kcinay055679
extract eslint format in seperated command
c3c7c06
@kcinay055679
readd prettier for css json and yaml
b2e97a6
@kcinay055679
update eslint config
c431c33
@kcinay055679
disable check for multipline tenary
8f1ba86
@kcinay055679
refactor config to extract rules to apply only on ts files
0ad5337
@kcinay055679
allow arguments
26910d7
@RandomTannenbaum @kcinay055679
Update excluded eslint rules that we want to resolve in follow up ticket
dfa9dd1
@RandomTannenbaum @kcinay055679
Fix sentence in readme
82a11a6
@RandomTannenbaum @kcinay055679
Allow ts-ignores
99f1a11
@Miguel7373 @kcinay055679
remove unnecessary comma
2f9cb03
@RandomTannenbaum @kcinay055679
Replace javax with jakarta in import order property of formatter
556f935
@kcinay055679
get all changes from apply branch
98abb4c
@kcinay055679
update precommit hook
fe974d3
@kcinay055679
update backend formatter
e3144d2
@kcinay055679
update java import order to alphabetically
935e6ae
kcinay055679 and others added 14 commits December 16, 2024 08:08
@kcinay055679 kcinay055679 force-pushed the feature/1135-formatter branch from 7d0c47e to 7116ba8 Compare December 17, 2024 12:09
@nevio18324 nevio18324 linked an issue Dec 17, 2024 that may be closed by this pull request
Re-enable ESLint rules #1209
Open
2 tasks
@kcinay055679 kcinay055679 force-pushed the feature/1135-formatter branch from b5572f9 to 40b83c3 Compare December 19, 2024 07:56
@kcinay055679 kcinay055679 force-pushed the feature/1135-formatter branch from c2c0b37 to 7734450 Compare December 20, 2024 12:09
Base automatically changed from feature/1135-formatter to main December 20, 2024 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kcinay055679 kcinay055679 Awaiting requested review from kcinay055679

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Re-enable ESLint rules
4 participants
@nevio18324 @kcinay055679 @Miguel7373 @RandomTannenbaum