apache / apache/polaris

Build/CI: Bring back (C) header checks

Open
#1,633 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
2.1k
Forks
522
Avg merge
1d 22h
Merged PRs (30d)
137

Description

### Describe the bug

#1562 removed the check for and automatic application of (C) headers.

It's too easy to have the wrong (C) header in files. Checks in CI and automatic application help a lot.

Spotless offers ways for what's mentioned on #1562 - for example `targetExcludeIfContentContains/Regex`.

### To Reproduce

_No response_

### Actual Behavior

_No response_

### Expected Behavior

_No response_

### Additional context

_No response_

### System information

_No response_

Contributor guide

Open the contributing guide

Research direction

Start by reviewing issue #1562 and the repository's current Spotless configuration to locate where header checks were removed. Compare Spotless's targetExcludeIfContentContains/Regex options with the prior behavior; done means CI checks and automatic (C) header application are restored without affecting intended exclusions.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system, ci-cd
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.