Skip to content

fix: prod deploy#10

Merged
priom merged 1 commit intoprodfrom
deploy/prod
Mar 31, 2026
Merged

fix: prod deploy#10
priom merged 1 commit intoprodfrom
deploy/prod

Conversation

@priom
Copy link
Copy Markdown
Member

@priom priom commented Mar 31, 2026

No description provided.

@priom priom self-assigned this Mar 31, 2026
@priom priom merged commit 89f93cb into prod Mar 31, 2026
3 checks passed
@priom priom deleted the deploy/prod branch March 31, 2026 18:27
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request configures the project for deployment on Cloudflare Pages by updating the Nitro preset, adding necessary permissions, and removing the deprecated @tanstack/react-router-ssr-query integration. Additionally, a patch was applied to @tanstack/start-plugin-core to fix a recursive CSS resolution issue. Regarding the review feedback, it is recommended to revert the formatter change in .vscode/settings.json back to Biome to ensure consistency across all JSON files and avoid unnecessary formatting diffs.

},
"[json]": {
"editor.defaultFormatter": "biomejs.biome"
"editor.defaultFormatter": "esbenp.prettier-vscode"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The default formatter for JSON has been changed to Prettier, which is inconsistent with the project's use of Biome for other file types (like jsonc). This inconsistency is likely what caused the package.json file to be reformatted from tabs to spaces, creating a noisy diff. It is recommended to use Biome for JSON files as well to maintain consistency across the codebase.

Suggested change
"editor.defaultFormatter": "esbenp.prettier-vscode"
"editor.defaultFormatter": "biomejs.biome"

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.

1 participant