Skip to content

feat: add Prettier integration (config only) #207

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

Merged
d-ulker merged 14 commits into from
Sep 29, 2025
Merged

feat: add Prettier integration (config only) #207

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
9211aa6
feat: add website assets from PR #169
d-ulker Sep 27, 2025
373da35
fix: replace HTML redirect favicon.ico with proper binary icon file
d-ulker Sep 27, 2025
97a6f63
fix: update HTML to use modular CSS structure
d-ulker Sep 27, 2025
ad48b66
fix: address code review issues in comprehensive-demo.css
d-ulker Sep 27, 2025
bdd5d57
feat: add core CSS architecture files
d-ulker Sep 27, 2025
b228a9e
feat: add UI component CSS files
d-ulker Sep 27, 2025
6095624
feat: add layout and interactive component CSS files
d-ulker Sep 27, 2025
1b8466f
feat: add final component CSS files and documentation
d-ulker Sep 27, 2025
3ba8ed0
feat: merge website assets with code review fixes
d-ulker Sep 27, 2025
ec81042
Merge branch 'main' of github.com:uelkerd/SAMO--DL
d-ulker Sep 27, 2025
ddf603f
Merge branch 'main' of github.com:uelkerd/SAMO--DL
d-ulker Sep 27, 2025
b3a46e5
Merge branch 'main' of github.com:uelkerd/SAMO--DL
d-ulker Sep 28, 2025
f18f155
feat: add Prettier integration without formatting existing files
d-ulker Sep 28, 2025
2de7c34
Resolve merge conflict: combine Prettier and test infrastructure
d-ulker Sep 29, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions website/.prettierrc
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{
"semi": true,
"trailingComma": "es5",
"singleQuote": true,
"printWidth": 80,
"tabWidth": 2,
"useTabs": false
}
2 changes: 2 additions & 0 deletions website/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@
"test:run": "vitest run",
"test:coverage": "vitest run --coverage",
"lint": "eslint js/**/*.js",
"format": "prettier --write .",
"format:check": "prettier --check .",
Comment on lines +12 to +13

@gemini-code-assist gemini-code-assist Bot Sep 28, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The Prettier scripts are currently targeting . which will attempt to format every file in the directory. While Prettier ignores some files by default (like node_modules), this is very broad and could format unintended files in the future (e.g., build artifacts, logs). It's better practice to use a more specific glob pattern to target only the files you intend to format.

    "format": "prettier --write \"**/*.{html,css,js,json,md}\"",
    "format:check": "prettier --check \"**/*.{html,css,js,json,md}\""

"build": "echo 'Static site - no build needed'"
},
"devDependencies": {
Expand Down
Loading