Skip to content

Fix/hikahana/unify format rules tamesu#1077

Merged
Wakai111 merged 6 commits intofix/wakape/unify-format-rulesfrom
fix/hikahana/unify-format-rules-tamesu
Mar 28, 2026
Merged

Fix/hikahana/unify format rules tamesu#1077
Wakai111 merged 6 commits intofix/wakape/unify-format-rulesfrom
fix/hikahana/unify-format-rules-tamesu

Conversation

@hikahana
Copy link
Copy Markdown
Collaborator

@hikahana hikahana commented Mar 28, 2026

対応Issue

resolve #0

概要

]

以下、codex生成

このブランチでやったことは2つです。

.vscode/settings.json を、VSCode 保存時の挙動が CLI とズレにくいように修正しました。具体的には source.organizeImports を外し、view/next-project の local Prettier を使うよう prettier.prettierPath と prettier.requireConfig、eslint.useFlatConfig を追加しています。

.github/workflows/auto-prettier.yml は、依存解決のブレを減らすために pnpm install --frozen-lockfile にし、さらに CI ログ確認の結果を踏まえて pnpm/action-setup のバージョンをローカル Docker の view と同じ 10.28.0 に固定しました。

調査としては、Docker 上の format / lint:fix と workflow のログを突き合わせて、workflow 側の pnpm が 10.33.0、ローカル Docker 側が 10.28.0 で一致していなかったことを確認しています。これが「ローカル Docker の lint:fix が workflow で打ち消される」原因候補として一番明確だったので、まずそこを揃えました。

画面スクリーンショット等

  • URL
    スクリーンショット

テスト項目

備考

@github-actions github-actions Bot added the bug Something isn't working label Mar 28, 2026
Copy link
Copy Markdown
Contributor

@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 updates the project's VS Code settings to enforce Prettier configurations and enable ESLint's flat config, while removing the automatic import organization on save. The majority of the code changes are automated formatting updates to JSX elements and Tailwind CSS class strings across numerous components. Feedback was provided regarding a specific formatting regression in SponsorActivitiesDesktopSection.tsx that reduces readability and is inconsistent with the rest of the PR's formatting style.

@hikahana hikahana changed the base branch from fix/wakape/unify-format-rules to develop March 28, 2026 11:17
@hikahana hikahana closed this Mar 28, 2026
@hikahana hikahana reopened this Mar 28, 2026
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Mar 28, 2026

Deploying finansu with  Cloudflare Pages  Cloudflare Pages

Latest commit: c3a97f3
Status: ✅  Deploy successful!
Preview URL: https://4a9590ef.finansu.pages.dev
Branch Preview URL: https://fix-hikahana-unify-format-ru.finansu.pages.dev

View logs

@hikahana hikahana changed the base branch from develop to fix/wakape/unify-format-rules March 28, 2026 11:41
@Wakai111 Wakai111 merged commit 62ead1a into fix/wakape/unify-format-rules Mar 28, 2026
1 check passed
@Wakai111 Wakai111 deleted the fix/hikahana/unify-format-rules-tamesu branch March 28, 2026 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants