Skip to content

chore: Add Prettier and pre-commit hook for consistent formatting, applying initial format fixes across documentation and examples. - #409

Merged
MJjainam merged 1 commit into
google-github-actions:mainfrom
parthasaradhie:ps/prettier-config
Dec 8, 2025
Merged

MJjainam merged 1 commit into
google-github-actions:mainfrom
parthasaradhie:ps/prettier-config

Conversation

@parthasaradhie

Copy link
Copy Markdown
Contributor

Adding prettier config as default formatter. and pre commit hook to format files automatically.
Reformatting existing file
This is part of #373

@parthasaradhie
parthasaradhie requested review from a team as code owners December 8, 2025 10:44
@parthasaradhie
parthasaradhie marked this pull request as draft December 8, 2025 10:48
…plying initial format fixes across documentation and examples.
@parthasaradhie
parthasaradhie marked this pull request as ready for review December 8, 2025 11:16
@MJjainam

MJjainam commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

@gemini-cli /review

@gemini-cli

gemini-cli Bot commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

🤖 Hi @MJjainam, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

@gemini-cli gemini-cli Bot left a comment

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.

## 📋 Review Summary

This pull request introduces Prettier to enforce consistent code formatting across the project, along with a pre-commit hook to automate the process. The changes are well-structured and significantly improve the codebase's consistency by reformatting numerous documentation and example files.

🔍 General Feedback

  • The introduction of Prettier is a great step towards maintaining a consistent style and reducing formatting-related noise in future contributions.
  • The configuration in .prettierrc is standard and the ignored files in .prettierignore are appropriate.
  • I've added one minor suggestion to the .husky/pre-commit script to align with shell scripting best practices.

Comment thread .husky/pre-commit
Comment thread .husky/pre-commit
@MJjainam
MJjainam enabled auto-merge (squash) December 8, 2025 11:34
@MJjainam
MJjainam merged commit 3127b84 into google-github-actions:main Dec 8, 2025
42 checks passed
parthasaradhie pushed a commit that referenced this pull request Dec 16, 2025
## What's Changed
* fix #371 use JSON output of gemini cli by @gsehgal in
#402
* fix: detect silent failures in gemini-cli by checking for invalid JSO…
by @megha1188 in
#406
* chore: Add Prettier and pre-commit hook for consistent formatting,
applying initial format fixes across documentation and examples. by
@parthasaradhie in
#409
* Rename `DEBUG` input to `GEMINI_DEBUG` and pass `--debug` flag to CLI.
by @parthasaradhie in
#407
* ci: auto update GitHub Actions using dependabot by @JamBalaya56562 in
#398
* Add job summary for Gemini CLI execution by @megha1188 in
#423

## New Contributors
* @megha1188 made their first contribution in
#406
* @parthasaradhie made their first contribution in
#409

**Full Changelog**:
v0.1.16...v0.1.17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants