FileList#<< bypasses exclusions while FileList#include respects them
まだ誰も着手していません。
評価
- 難易度
- 1/5
- 見積もり時間
- 1時間未満
- 初心者へのやさしさ
- 45/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- ruby
- 領域
- build-system
調査の方向性
Start with FileList#<< and compare its exclusion handling with FileList#include and resolve_exclude. Reproduce the abc.c example and verify that a file excluded before being appended remains absent after resolving; confirm the associated fix and tests pass.
索引モデルが issue の本文から書いたものです。
説明
Problem
The << and include methods behave inconsistently with exclusions:
excludetheninclude: exclusion wins (correct)excludethen<<: file gets included anyway (bug)
Example
include_list = FileList.new
include_list.exclude 'abc.c'
include_list.include 'abc.c' # Correctly excluded: []
shovel_list = FileList.new
shovel_list.exclude 'abc.c'
shovel_list << 'abc.c' # Bug: returns ["abc.c"]
shovel_list.resolve # Still returns ["abc.c"] - resolve doesn't fix it
shovel_list.include 'xyz.rb' # Now returns ["xyz.rb"], "abc.c" is finally removed
Fix
Add resolve_exclude to the << method:
def <<(obj)
resolve
@items << Rake.from_pathname(obj)
resolve_exclude # ADD THIS LINE
self
end
I have a PR with fix & tests ready to go if this approach looks good to maintainers
- 主要言語
- Ruby
- スター
- 2.5k
- フォーク
- 650
- 平均マージ
- 6分
- マージ済み PR（30日）
- 3
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ruby/rake のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 28/100
-
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
TheOdinProject/curriculum#31417 · コメント 2 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
glossarist/glossarist-ruby#238 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
palladius/rails8-app-on-gcp#145 ·