apache / apache/pinot

Add pre-commit hooks to the repository

Open
#8,349 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
6.1k
Forks
1.5k
Avg merge
2d 3h
Merged PRs (30d)
195

Description

Currently, the only way to force linting in a PR is via Github Action scripts. However, with so many PRs being raised we should be using pre-commit hooks so that these linting checks run before doing a commit.
This will help keep the git tree much cleaner by removing the need for all the commits just doing linting fixes.

Contributor guide

Open the contributing guide

Research direction

Start by inspecting the repository's GitHub Action scripts and existing linting checks to understand what currently runs for pull requests. Determine how those checks should be represented as pre-commit hooks and verify that the hooks run before commits while the existing PR validation remains effective. Done means the repository documents and supports the new hooks without lint-only follow-up commits.

Written by the indexing model from the issue text.

Assessment

Tech stack
git, github-actions, java
Domain
ci-cd, developer-experience, tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.