Skip to content

🔧 add prek for world peace - #363

Merged
ianardee merged 1 commit into
mainfrom
prek
Oct 8, 2026
Merged

ianardee merged 1 commit into
mainfrom
prek

Conversation

@ianardee

@ianardee ianardee commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

Description

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires a change to the official Guide documentation.

@ianardee
ianardee requested review from sebastianMindee and a balanced review from Copilot October 8, 2026 09:55

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The prek configuration is invalid for standard TOML parsers, and the token-enabled third-party action uses a mutable tag.

2 open findings
What changed in this PR

Introduces prek hooks for formatting, validation, and leak detection while refreshing CI actions.

Changes:

  • Adds Betterleaks and Maven-based prek hooks.
  • Integrates Betterleaks into static analysis.
  • Upgrades GitHub Actions and removes trailing whitespace.
File Description
prek.toml Configures development hooks.
.editorconfig Adds TOML formatting rules.
.github/​workflows/​_static-analysis.yml Adds Betterleaks and upgrades actions.
.github/​workflows/​_build.yml Upgrades build actions.
.github/​workflows/​_codeql.yml Upgrades checkout.
.github/​workflows/​_publish-code.yml Upgrades publishing actions.
.github/​workflows/​_publish-docs.yml Upgrades documentation actions.
.github/​workflows/​_test-cli.yml Upgrades CLI test actions.
.github/​workflows/​_test-code-samples.yml Upgrades sample test actions.
.github/​workflows/​_test-integrations.yml Upgrades integration test actions.
PageOptions.java Removes Javadoc trailing whitespace.
CHANGELOG.md Removes trailing whitespace.

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

Comment thread .github/workflows/_static-analysis.yml Outdated
Comment thread prek.toml

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

Betterleaks findings cannot currently fail CI, allowing newly introduced credentials to pass required checks.

1 open finding
2 resolved since last review

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

Comment thread .github/workflows/_static-analysis.yml
@ianardee
ianardee merged commit 2c11160 into main Oct 8, 2026
15 checks passed
@ianardee
ianardee deleted the prek branch October 8, 2026 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants