Skip to content

Exclude reviewed root documents from the Java PR pipeline - #50474

Merged
vcolin7 merged 3 commits into
mainfrom
vcolin7-root-documentation-exclusions
Sep 16, 2026
Merged

vcolin7 merged 3 commits into
mainfrom
vcolin7-root-documentation-exclusions

Conversation

@vcolin7

@vcolin7 vcolin7 commented Sep 15, 2026

Copy link
Copy Markdown
Member

Description

Changes limited to the following root documents don't need the unified Java PR pipeline, so this PR adds them to the exclusions list:

  • AGENTS.md
  • CODE_OF_CONDUCT.md
  • CONTRIBUTING.md
  • LICENSE.txt
  • NOTICE.txt
  • README.md
  • SECURITY.md
  • SUPPORT.md

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes. Not applicable: CI tooling only.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 2 pipeline(s).
34 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No unresolved blocking issues were identified.

Pull request overview

This PR excludes reviewed root documentation files from the Java PR pipeline while preserving package-level validation.

Changes:

  • Adds root-document trigger and package-selection exclusions.
  • Adds regression tests for exclusion and mixed-PR behavior.
  • Documents the updated pipeline behavior.
File summaries
File Description
eng/scripts/tests/PullRequest-Trigger.tests.ps1 Tests exclusion and mixed-PR behavior.
eng/README.md Documents pipeline rules and validation commands.
eng/pipelines/pullrequest.yml Adds mirrored root-document exclusions.
Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@vcolin7
vcolin7 merged commit 003b73a into main Sep 16, 2026
22 checks passed
@vcolin7
vcolin7 deleted the vcolin7-root-documentation-exclusions branch September 16, 2026 22:43
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