lint: add tests for three caption rules that have none
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 85/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- bun, typescript
- 領域
- testing-qa
調査の方向性
3つのルールは packages/lint/src/rules/captions.ts の90行目、171行目、212行目で定義されています。テストは packages/lint/src/rules/captions.test.ts に既存のテストと並べて追加してください。まず各ルールの条件を読み、そのルールを発動させる構成と、発動しないはずの近い変形を1つずつ書き、コードと重大度を検証します。caption_transcript_not_inline、caption_textshadow_on_group_container、caption_fittext_scale_mismatch のそれぞれに肯定テストと否定テストがあり、ルールを変更せずに bun run --cwd packages/lint test が通れば完了です。
索引モデルが issue の本文から書いたものです。
説明
Problem
These lint rules have no test that mentions their code, so a change that breaks them would go unnoticed:
| Code | Severity | Defined at |
|---|---|---|
caption_transcript_not_inline |
error | packages/lint/src/rules/captions.ts:90 |
caption_textshadow_on_group_container |
warning | packages/lint/src/rules/captions.ts:171 |
caption_fittext_scale_mismatch |
warning | packages/lint/src/rules/captions.ts:212 |
Tests go in packages/lint/src/rules/captions.test.ts.
Proposed solution
Add tests to the matching *.test.ts file next to each rule, following the style of the other tests in that file. For each rule, add at least:
- one composition that triggers the finding, asserting the code and severity, and
- one similar composition that should not trigger it.
Read the rule's code first: the condition it checks tells you what the minimal triggering composition looks like.
Acceptance criteria
- Each code above is asserted by at least one positive and one negative test.
bun run --cwd packages/lint testpasses.- No rule behaviour changes. If writing a test shows a rule misbehaving, note it in the PR or open a separate issue rather than changing the rule in the same PR.
- 主要言語
- TypeScript
- スター
- 54.1k
- フォーク
- 4.9k
- 平均マージ
- 8時間 49分
- マージ済み PR（30日）
- 667
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
heygen-com/hyperframes のほかの issue
-
product-launch-video: assemble-index.mjs is not idempotent — a second run drops every approved frame video対応中かも @Dante-dan が 1 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
heygen-com/hyperframes#5733 ·
メンテナーはふだん 1 日以内に返信
-
gsap_tween_tail counts GSAP's force3D and autoRound flags as properties対応中かも @rudycelekli が 1 日前に担当しました。 オープンbug triage/needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
heygen-com/hyperframes#5683 ·
メンテナーはふだん 1 日以内に返信
-
Timeline: show a clip's full name when its label is cut off対応中かも @DanielMcSheehy が今日担当しました。 オープンdesktop-candidate difficulty/easy enhancement good first issue help wanted triage/ready
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
heygen-com/hyperframes#5582 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Studio: press ? to open the keyboard shortcuts panel対応中かも @B-Payal が 4 日前に担当しました。 オープンdesktop-candidate difficulty/easy enhancement good first issue help wanted triage/ready
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
heygen-com/hyperframes#5580 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
remotion-to-hyperframes: Easing.cubic is mapped to power3 (quartic); cubic is power2対応中かも @user-github-me が 4 日前に担当しました。 オープンdifficulty/easy documentation help wanted triage/ready
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
heygen-com/hyperframes#5538 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
heygen-com/hyperframes の issue をすべて見る
似ている issue
-
The inspector never shows the instances field, so scaling out is unreachable対応中かも @Mrkewl が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
xevrion/breakscale#96 ·
メンテナーはふだん 4 日以内に返信
-
Object parameters lose their structure when resources is enabled対応中かも @asts-top が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
rossant/awesome-math#199 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
jaegertracing/jaeger-ui#4618 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
mifi/lossless-cut#3125 ·
メンテナーはふだん 1 日以内に返信