FileList#<< bypasses exclusions while FileList#include respects them

Abierto

#637 abierto el 5/6/2025 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad: 1/5
Tiempo estimado: Menos de una hora
Aptitud para principiantes: 45/100

Tipo de issue: Error
Claridad: Bien especificado
Estado de actividad: Estancado
Stack tecnológico: ruby
Área: build-system

Línea de trabajo

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.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Problem

The << and include methods behave inconsistently with exclusions:

exclude then include: exclusion wins (correct)
exclude then <<: 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

Lenguaje dominante: Ruby
Estrellas: 2.5k
Forks: 650
Merge medio: 6 min
PR fusionados (30 d): 3

Guía de contribución

Abrir la guía de contribución

Primeros pasos

Lee el issue completo y luego la guía de contribución del proyecto.
Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
Haz un fork del repositorio y trabaja en una rama.
Abre un pull request que haga referencia al número del issue.

Más de ruby/rake

rake -P output is ambiguous with prereq namespaces Abierto

Dificultad 3/5 1-2 días Aptitud para principiantes 68/100

ruby/rake#704 · 1 comentario · Abierto el 18/3/2026
--dry-run (or -n) doesn't seem to trigger `when_writing` Abierto

Dificultad 3/5 1-2 días Aptitud para principiantes 48/100

ruby/rake#691 · Abierto el 5/2/2026
Feature request: corollary to `sh` that captures stdout Abierto

Dificultad 5/5 Más de una semana Aptitud para principiantes 28/100

ruby/rake#678 · 241 comentarios · Abierto el 16/12/2025
Rule dependencies' behaviour is strange, possibly broken or poorly documented Abierto

Dificultad 4/5 3-5 días Aptitud para principiantes 38/100

ruby/rake#626 · Abierto el 7/5/2025
Task Options in Command-Line Abierto

Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100

ruby/rake#598 · 1 comentario · Abierto el 14/10/2024

Todos los issues de ruby/rake

Issues similares

P4 FR: Add one-click copy-to-clipboard button to code snippets in static workshop HTML Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

palladius/rails8-app-on-gcp#145 · Abierto el 23/9/2026
Restore `preview_sensitive_media` to the admin UI Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

mastodon/mastodon#40662 · Abierto el 23/9/2026
Cop idea: prefer `be_an_instance_of` over `be_instance_of` Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

rubocop/rubocop-rspec#2236 · Abierto el 23/9/2026
condition solver and JSON Schema are out of sync? Abierto

bug

Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100

riscv/riscv-unified-db#2624 · 1 reacción · Abierto el 23/9/2026
Bounces#match crashes with NameError for already-bounced recipients Abierto

Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100

yegor256/mailanes#322 · Abierto el 22/9/2026

Más issues de Ruby