Skip to content

Avoid logging of SBRP reference-only packages in incremental builds#4020

Merged
ViktorHofer merged 2 commits intomainfrom
ViktorHofer-patch-1
Jan 5, 2026
Merged

Avoid logging of SBRP reference-only packages in incremental builds#4020
ViktorHofer merged 2 commits intomainfrom
ViktorHofer-patch-1

Conversation

@ViktorHofer
Copy link
Member

The LogReferenceOnlyPackages target always runs, even in incremental builds that skip the repo build. Fix this by merging the targets and applying inputs and outputs.

The LogReferenceOnlyPackages target always runs, even in incremental builds that skip the repo build. Fix this by merging the targets and applying inputs and outputs.
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 aims to prevent the LogReferenceOnlyPackages target from running unnecessarily during incremental builds. It consolidates the logging functionality into the CopyReferenceOnlyPackages target and implements incremental build support using Inputs and Outputs attributes.

Key changes:

  • Merged LogReferenceOnlyPackages target into CopyReferenceOnlyPackages target
  • Added Inputs and Outputs attributes to enable incremental build tracking
  • Changed item group name from _ReferenceOnlyPackages to ReferenceOnlyPackage for consistency

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI commented Jan 2, 2026

@ViktorHofer I've opened a new pull request, #4039, to work on those changes. Once the pull request is ready, I'll request review from you.

@ViktorHofer ViktorHofer merged commit fb2e783 into main Jan 5, 2026
9 of 11 checks passed
@ViktorHofer ViktorHofer deleted the ViktorHofer-patch-1 branch January 5, 2026 18:58
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