apache/sedona

Optimize shellcheck ignores for actionlint pre-commit

Open

#2 738 ouverte le 12 mars 2026

Voir sur GitHub
 (9 commentaires) (0 réactions) (0 assignés)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 ??

Guide contributeur

Stack technique
shellgithub actions
Domaine
devopstooling
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic knowledge of shell scriptsFamiliarity with pre commit hooks
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75
Direction de recherche
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.