Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

chore(tooling): adoptar rustfmt con gate de CI de formato (baseline no fmt-clean)

オープン
#184 コメント 3 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@SrStamm がすでに取り組んでいます。

2026年10月25日 から。

評価

この issue はまだ評価されていません。

説明

Pre-flight checks
  • I searched existing issues and found no duplicate.
  • I understand that this proposal requires product review before implementation.
Problem to solve

El repo no tiene una política de formato Rust verificada:

  • No existe rustfmt.toml en ninguna parte del repo.
  • Ningún workflow de CI corre cargo fmt --check (revisados los 9 de .github/workflows/).
  • El baseline no está fmt-clean: cargo fmt --check falla hoy (p. ej. src-tauri/src/wm_compat.rs y src-tauri/src/cli_process.rs tienen líneas largas sin formato).

Consecuencia observada en la práctica (PR #183, ciclo de revisión de #181): un cargo fmt -- <archivos> durante el trabajo generó ruido de formato en 29 archivos que no se tocaban (~3.700 líneas), que hubo que descartar para no inflar el diff de un PR de bugfix. Cada contribuidor decide su estilo y cualquier PR que mezcle cambios funcionales con formateo mata la revisión del diff.

Proposed solution

Tres pasos, en este orden:

  1. Decidir configuración (opcional): si se quiere algo distinto del default de rustfmt (p. ej. max_width más generoso; varios tests actuales se escribieron con líneas >100 columnas y rustfmt los parte), definirlo en rustfmt.toml antes de formatear.
  2. Un commit de formateo único y aislado: chore(style): cargo fmt solo formato, sin mezclar con nada funcional, revisado y aterrizado solo. Así ese diff grande se lee una vez y no vuelve a aparecer.
  3. Gate de CI: cargo fmt --check como paso en ci.yml / linux-pr-ci.yml (coste en segundos) para que el formato sea requisito de PR y no decisión de cada contribuidor.

Coordinación: aterrizar después del merge de #183 (PR en vuelo) para no rebasear sobre un reformateo masivo.

Affected area

CI or automation

Scope and non-goals

Incluye: formateo de los targets Rust (src-tauri), posible rustfmt.toml, gate cargo fmt --check en CI.

No incluye: cambios de comportamiento o refactors (el formateo es solo formato), política de warnings de clippy ni otros linters, formateo del frontend (prettier/eslint, si aplica, sería propuesta aparte), ni cambios al flujo de #183/#181.

Alternatives considered
  • No hacer nada / discreción por contribuidor: lleva a estilos mezclados y PRs ruidosos (observado en la práctica).
  • Formatear solo archivos tocados por PR: como el baseline no está fmt-clean, sigue generando ruido en el diff.
  • Gate de CI sin el commit de formateo: el check fallaría de inmediato en el baseline; no es viable sin el paso 2.
主要言語
TypeScript
スター
10
フォーク
2
平均マージ
4時間 44分
マージ済み PR(30日)
100

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

egdev6/quorb のほかの issue

egdev6/quorb の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。