apache/sedona

Optimize shellcheck ignores for actionlint pre-commit

Open

#2.738 aberto em 12 de mar. de 2026

Ver no GitHub
 (9 comments) (0 reactions) (0 assignees)Scala (1.953 stars) (693 forks)batch import
github_actionsgood first issuepre-commitshell

Description

Original failed run seen here:

https://github.com/apache/sedona/actions/runs/22992869182/job/66757741342

This hook is here:

https://github.com/apache/sedona/blob/3ac639748416b8e6975a409e53bb571bc36a6343/.pre-commit-config.yaml#L510

So maybe we can remove some ignores and clean up the code ??

Guia do colaborador

Pilha de tecnologia
shellgithub actions
Domain
devopstooling
Tipo Issue
refactor
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.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Basic knowledge of shell scriptsFamiliarity with pre commit hooks
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75
Direção de pesquisa
Examine the linked failed run and the specific shellcheck ignore lines in .pre commit config.yaml. Identify which shellcheck rules can be enabled without causing failures, and consider refactoring the shell scripts to comply with those rules. Review comments for any suggestions.