Skip to content

Conversation

@JDarDagran
Copy link
Contributor

Problem

prek is quite fresh replacement written in Rust for pre-commit. It promises to be way faster and use much less disk resources.

Solution

This requires some cosmetic changes in CI and docs.


SPDX-License-Identifier: Apache-2.0
Copyright 2018-2025 contributors to the OpenLineage project

@JDarDagran JDarDagran requested a review from a team as a code owner August 12, 2025 14:59
@boring-cyborg boring-cyborg bot added area:ci CI area:documentation Improvements or additions to documentation labels Aug 12, 2025
@boring-cyborg
Copy link

boring-cyborg bot commented Aug 12, 2025

Thanks for opening your first OpenLineage pull request! We appreciate your contribution. If you haven't already, please make sure you've reviewed our guide for new contributors (https://github.com/OpenLineage/OpenLineage/blob/main/CONTRIBUTING.md).

@JDarDagran JDarDagran requested review from Copilot and removed request for a team August 12, 2025 15:00
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR replaces pre-commit with prek, a Rust-based alternative that promises faster performance and reduced disk usage. The changes update references throughout the codebase to use the new tool while maintaining the same functionality.

  • Replace pre-commit with prek in documentation and CI configuration
  • Update installation and usage commands across contributing guidelines and schema documentation
  • Bump CI cache version to reflect the tooling change

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
website/docs/spec/schemas.md Updates installation and run commands from pre-commit to prek
CONTRIBUTING.md Updates pre-commit setup instructions to use prek instead
.circleci/continue_config.yml Updates CI configuration to install and run prek, bumps cache version

Signed-off-by: Jakub Dardzinski <kuba0221@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented Aug 12, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.17%. Comparing base (78c4517) to head (6567c27).
⚠️ Report is 44 commits behind head on main.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3965   +/-   ##
=======================================
  Coverage   85.17%   85.17%           
=======================================
  Files          57       57           
  Lines        3844     3844           
=======================================
  Hits         3274     3274           
  Misses        570      570           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JDarDagran JDarDagran merged commit a653df8 into main Aug 13, 2025
70 checks passed
@JDarDagran JDarDagran deleted the ci/use-prek branch August 13, 2025 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ci CI area:documentation Improvements or additions to documentation area:integration/dbt openlineage-dbt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants