Skip to content

Internal: Setup prettier and fix all lint/format issues#671

Open
rami-elementor wants to merge 3 commits into
mainfrom
prettier
Open

Internal: Setup prettier and fix all lint/format issues#671
rami-elementor wants to merge 3 commits into
mainfrom
prettier

Conversation

@rami-elementor

@rami-elementor rami-elementor commented Jun 25, 2026

Copy link
Copy Markdown
Member

✨ PR Description

1. Problem & Context

Integrate Prettier code formatter across the codebase to enforce consistent code style and reduce lint/format friction. Replaces manual eslint-based formatting with Prettier's opinionated, deterministic approach.

2. What Changed (Where)

  • Config & tooling: Added .prettierrc, updated package.json (Prettier + eslint-config-prettier + eslint-plugin-prettier), updated .eslintrc.js to remove conflicting rules and extend Prettier config
  • TypeScript/React files (60+ files in modules/admin-home, dev/js, tests/playwright): Reformatted spacing, arrow functions, brackets, template literals, JSX attributes
  • Build config (webpack.config.js): Improved readability with expanded multi-line declarations

3. How It Works

Prettier enforces formatting via .prettierrc config (tabs, 80-char width, always arrow parens, bracket spacing). ESLint integration delegates formatting concerns to Prettier via eslint-plugin-prettier and eslint-config-prettier (disables conflicting ESLint rules). npm run format now triggers Prettier instead of manual lint-fix cycles.

4. Risks

Low risk—purely formatting, no logic changes. Only concern: Prettier's 80-char line width may force awkward breaks in long strings/URLs (e.g., test selectors), but this improves consistency at minor verbosity cost. Existing tests/functionality unaffected.

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

@wiz-9a149474ff

Copy link
Copy Markdown

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities -
Data Finding Sensitive Data -
Secret Finding Secrets -
IaC Misconfiguration IaC Misconfigurations -
SAST Finding SAST Findings 3 Low
Software Management Finding Software Management Findings -
Total 3 Low

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant