fix: byline のぶんラチェットを上げる(main が赤い) - #72
Merged
Merged
Conversation
`main` が lint で赤い。#66 のマージで 2 つのファイルが、その直前に打たれた 杭を越えた: `src/publishChecks.ts` 1301 → 1303、 `test/test_publishChecks.ts` 1500 → 1533。 **これはラチェットが設計どおりに働いた形なので、上げるのが正しい対応。** そこのコメント自身が「分割は移動ではなく本当の変更で、publishChecks の refusal は家族ごとに並んでいて、テストは各 assertion をその家族の隣に置いて いる」と書いている。杭は成長を禁じるものではなく、**成長を誰かが承認する diff の 1 行にする**もの。だから伸ばしたのと同じ変更で数字を上げ、何が 伸ばしたのかを書いた。 **PR では赤くならなかった理由も書いた。** #67 がファイルの実測値で杭を打った とき、#66 はその手前のベースに対して既に緑だった。どちらもおかしくない —— 機能の隣にラチェットが着地したときの普通の形。
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR fixes main's lint failure by intentionally raising the two max-lines thresholds to the measured sizes introduced by article.byline support, with comments documenting the ratchet workflow, the merge-order interaction, and why splitting the files is not the intended fix. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
isamu
approved these changes
Aug 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
mainが lint で赤いので、その修正です。#66 のマージで 2 つのファイルが、その直前に打たれた杭を越えました:
src/publishChecks.tstest/test_publishChecks.tsbylineが「記事の描画欄を列挙している 2 つのリスト」に加わり、自分の refusal を連れてきたぶんです。上げるのが正しい対応です
そこのコメント自身がこう書いています:
杭は成長を禁じるものではなく、成長を誰かが承認する diff の 1 行にするものです。だから伸ばしたのと同じ変更で数字を上げ、何が伸ばしたのかをコメントに書きました。分割はここでは筋が悪い、というのは #67 が既に判断済みです。
なぜ PR では赤くならなかったか
#67 がファイルの実測値で杭を打ったとき、#66 はその手前のベース(
4722a99)に対して既に緑でした。二つの変更が一度も出会っていないだけで、どちらもおかしくありません —— 機能の隣にラチェットが着地したときの普通の形です。これもコメントに残しました。確認
lint 0 error / 9 warnings、
lint:overridesは 14 個の抑制がすべて何かを抑えている状態のまま、typecheck 0、533 pass / 0 fail、check:appsは実在 10 アプリとも PASS。Summary by Sourcery
Raise the publish-check line-count ratchets to accommodate the article byline additions and keep main lint green.
Bug Fixes:
Enhancements: