Skip to content

fix: prod deploy - #10

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

fix: prod deploy#10
priom merged 1 commit into
prodfrom
deploy/prod

Conversation

@priom

@priom priom commented Mar 31, 2026

Copy link
Copy Markdown
Member

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

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

Copy link
Copy Markdown

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.

Comment thread .vscode/settings.json
},
"[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