Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
9 changes: 9 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -140,6 +140,15 @@ docs/website/node_modules/
# package-lock.json is ignored because we use pnpm exclusively (pnpm-lock.yaml is committed)
docs/website/package-lock.json

# Web Apps (Astro, Vite, etc.)
# Astro cache/build directories
**/.astro/
clients/web/apps/**/dist/
clients/web/apps/**/node_modules/
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.

🧹 Nitpick | 🔵 Trivial

Redundant pattern: already covered by **/node_modules/ on line 92.

The global **/node_modules/ pattern already matches all node_modules/ directories at any depth, making this more specific pattern unnecessary.

🤖 Prompt for AI Agents
Before applying any fix, first verify the finding against the current code and
decide whether a code change is actually needed. If the finding is not valid or
no change is required, do not modify code for that item and briefly explain why
it was skipped.
In @.gitignore at line 147, Remove the redundant gitignore entry
"clients/web/apps/**/node_modules/" because it is already covered by the broader
"**/node_modules/" pattern; delete the specific pattern line and keep only the
global "**/node_modules/" entry to avoid duplication.


# Symlinks auto-generados por Gradle
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.

⚠️ Potential issue | 🟡 Minor

Inconsistent comment language.

This comment is in Spanish while the rest of the file uses English. Consider aligning for consistency:

Proposed fix
-# Symlinks auto-generados por Gradle
+# Symlinks auto-generated by Gradle
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
# Symlinks auto-generados por Gradle
# Symlinks auto-generated by Gradle
🤖 Prompt for AI Agents
Before applying any fix, first verify the finding against the current code and
decide whether a code change is actually needed. If the finding is not valid or
no change is required, do not modify code for that item and briefly explain why
it was skipped.
In @.gitignore at line 149, The comment "# Symlinks auto-generados por Gradle"
in the .gitignore is in Spanish while the rest of the file is English; update
that comment to English for consistency (e.g., change the line containing
"Symlinks auto-generados por Gradle" to "Gradle auto-generated symlinks" or
similar), preserving the comment marker and placement.

/docs

# Spotless Prettier Node Modules (auto-generated by build)
build/spotless-prettier-node-modules-*/
build/spotless-prettier-node-modules-*/node_modules/
Expand Down
360 changes: 0 additions & 360 deletions apps/iosApp/iosApp.xcodeproj/project.pbxproj

This file was deleted.

This file was deleted.

Loading
Loading