Skip to content

Conversation

@artembelik
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings November 5, 2025 13:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR simplifies the Prettier configuration by leveraging Prettier's built-in defaults and using glob patterns for better maintainability.

  • Simplified the prettier script command to check all files instead of explicitly listing file extensions
  • Removed redundant ignore patterns that Prettier handles by default (like node_modules/, .angular/, etc.)
  • Updated ignore patterns to use glob syntax and made them more specific (e.g., CHANGELOG.md**/CHANGELOG.md)

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
package.json Simplified prettier script to check all files (.) instead of explicitly listing file extensions
.prettierignore Removed default ignore patterns, converted to glob patterns, and added comments for better organization

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@artembelik artembelik self-assigned this Nov 5, 2025
@github-actions
Copy link

github-actions bot commented Nov 5, 2025

Visit the preview URL for this PR (updated for commit 5f4e373):

https://koobiq-next--prs-1099-i1tob7zl.web.app

(expires Mon, 10 Nov 2025 13:40:40 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@artembelik artembelik merged commit 777bdab into main Nov 6, 2025
8 checks passed
@artembelik artembelik deleted the chore/prettier-all-files branch November 6, 2025 08:14
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.

3 participants