bazelbuild/bazel

Add support for comments in .bazelignore files

Open

#19.821 aberto em 15 de out. de 2023

Ver no GitHub
 (0 comments) (1 reaction) (0 assignees)Java (4.465 forks)batch import
P2help wantedteam-OSStype: feature request

Métricas do repositório

Stars
 (25.384 stars)
Métricas de merge de PR
 (Mesclagem média 22d 20h) (77 fundiu PRs em 30d)

Description

Description of the feature request:

It would be nice if it would be possible to use comments in .bazelignore files, e.g.

ree/acceptance_tests/ # has its own MODULE.bazel file
third_party # ignore third-party folder

Currently, line 1 is interpreted by Bazel as "ignore folder that is named ree/acceptance_tests/ # has its own MODULE.bazel file", i.e. it considers the comment as a part of the path name to ignore.

Since # works as a comment in BUILD(.build), WORKSPACE(.bazel), MODULE.bazel, .bazelrc etc. it would be nice if it would also work for .bazelignore files

What underlying problem are you trying to solve with this feature?

Be able to write down some description why something is ignored

Which operating system are you running Bazel on?

macOS, Windows, Ubuntu

What is the output of bazel info release?

8.5.0

Guia do colaborador

Direção de pesquisa
Examine o código fonte do Bazel para a análise do arquivo .bazelignore. Identifique onde as linhas são lidas e os caminhos extraídos. Adicione lógica para remover comentários (começando com #) após cortar espaços em branco. Garanta compatibilidade com o comportamento existente e adicione testes.
Pilha de tecnologia
java
Domain
build system
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75