Skip to content

Conversation

@ogenstad
Copy link
Contributor

@ogenstad ogenstad commented Nov 28, 2025

Summary by CodeRabbit

  • Chores
    • Enhanced CI/CD validation to ensure build integrity.

✏️ Tip: You can customize this high-level summary in your review settings.

@github-actions github-actions bot added the group/ci Issue related to the CI pipeline label Nov 28, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 28, 2025

Walkthrough

A new CI job named uv-lock-check was added to the GitHub Actions workflow configuration. This job triggers when uv-related files or workflow definitions change, and depends on the files-changed and prepare-environment jobs. The job runs on ubuntu-latest with a 5-minute timeout, checks out the code, sets up Python 3.12, installs UV via the astral-sh/setup-uv action using the UV_VERSION variable from the prepare-environment job, and validates the pyproject.toml and uv.lock files offline. No existing jobs or workflow logic were modified.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately and concisely describes the main change: adding validation for uv.lock in the CI workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch pog-check-uv-lock

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0ba9ef2 and 4bcfc7e.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Cloudflare Pages
🔇 Additional comments (1)
.github/workflows/ci.yml (1)

122-143: I'll help you verify the concerns in this review comment. Let me examine the workflow files to check the claims about setup-uv versions and the UV_VERSION output.
<function_calls>


#!/bin/bash

Find all workflow files

echo "=== Workflow files in .github/workflows/ ==="
ls -la .github/workflows/ 2>/dev/null || echo "Directory not found"

echo -e "\n=== Checking ci.yml for astral-sh/setup-uv versions ==="
grep -n "astral-sh/setup-uv" .github/workflows/ci.yml

echo -e "\n=== Checking for UV_VERSION output across workflow files ==="
grep -r "UV_VERSION" .github/workflows/


</function_calls>


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cloudflare-workers-and-pages
Copy link

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4bcfc7e
Status: ✅  Deploy successful!
Preview URL: https://2c1b73fd.infrahub-sdk-python.pages.dev
Branch Preview URL: https://pog-check-uv-lock.infrahub-sdk-python.pages.dev

View logs

@codecov
Copy link

codecov bot commented Nov 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@           Coverage Diff            @@
##           develop     #665   +/-   ##
========================================
  Coverage    75.57%   75.57%           
========================================
  Files          113      113           
  Lines         9508     9508           
  Branches      1447     1447           
========================================
  Hits          7186     7186           
  Misses        1835     1835           
  Partials       487      487           
Flag Coverage Δ
integration-tests 34.96% <ø> (ø)
python-3.10 48.88% <ø> (+0.02%) ⬆️
python-3.11 48.86% <ø> (-0.03%) ⬇️
python-3.12 48.86% <ø> (ø)
python-3.13 48.86% <ø> (+0.02%) ⬆️
python-3.9 47.56% <ø> (ø)
python-filler-3.12 24.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ogenstad ogenstad marked this pull request as ready for review November 28, 2025 10:16
@ogenstad ogenstad requested a review from a team November 28, 2025 10:16
@ogenstad ogenstad merged commit 983da61 into develop Nov 28, 2025
21 checks passed
@ogenstad ogenstad deleted the pog-check-uv-lock branch November 28, 2025 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

group/ci Issue related to the CI pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants