Skip to content

Conversation

@paulmedynski
Copy link
Contributor

@paulmedynski paulmedynski commented Nov 18, 2025

Description

  • Changed CI pipelines to trigger on PRs, Commits, and Schedules.
  • Added some repo config files to manage code ownership and review controls.

- Changed CI pipelines to trigger on PRs, Commits, and Schedules.
Copilot AI review requested due to automatic review settings November 18, 2025 19:35
Copilot finished reviewing on behalf of paulmedynski November 18, 2025 19:35
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 reconfigures CI pipelines for the 6.0 release branch to properly handle pull requests, commits, and scheduled runs. The changes separate PR validation from commit-based CI triggers and establish distinct scheduled runs for GitHub and ADO branches.

Key Changes:

  • Added dedicated PR trigger configuration to validate changes against the release/6.0 branch
  • Separated commit triggers for release/6.0 and internal/release/6.0 branches with batch processing
  • Updated scheduled runs from Saturday to Sunday with separate schedules for GitHub (release/6.0) and ADO (internal/release/6.0) branches

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
eng/pipelines/dotnet-sqlclient-ci-project-reference-pipeline.yml Adds PR triggers for release/6.0, separates commit triggers for both release branches, and schedules Sunday runs at 06:00 UTC (GitHub) and 07:00 UTC (ADO)
eng/pipelines/dotnet-sqlclient-ci-package-reference-pipeline.yml Adds PR triggers for release/6.0, separates commit triggers for both release branches, and schedules Sunday runs at 06:30 UTC (GitHub) and 07:30 UTC (ADO)

@paulmedynski paulmedynski requested a review from a team November 18, 2025 19:50
mdaigle
mdaigle previously approved these changes Nov 18, 2025
Copilot AI review requested due to automatic review settings November 18, 2025 23:50
Copilot finished reviewing on behalf of paulmedynski November 18, 2025 23:52
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

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

@codecov
Copy link

codecov bot commented Nov 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.82%. Comparing base (1542afd) to head (bc6149a).
⚠️ Report is 3 commits behind head on release/6.0.

Additional details and impacted files
@@               Coverage Diff               @@
##           release/6.0    #3780      +/-   ##
===============================================
+ Coverage        75.51%   75.82%   +0.30%     
===============================================
  Files              244      244              
  Lines            40212    40221       +9     
===============================================
+ Hits             30368    30498     +130     
+ Misses            9844     9723     -121     
Flag Coverage Δ
addons 92.58% <ø> (ø)
netcore 75.69% <ø> (+0.30%) ⬆️

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

☔ 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.

@paulmedynski paulmedynski merged commit 50b1436 into release/6.0 Nov 20, 2025
240 checks passed
@paulmedynski paulmedynski deleted the dev/paul/release/6.0/ci-pipelines branch November 20, 2025 20:30
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.

4 participants