Relax Java pipeline path base length to 30 - #50462
Merged
Weidong Xu (weidongxu-microsoft) merged 1 commit intoSep 15, 2026
Merged
Weidong Xu (weidongxu-microsoft) merged 1 commit into
Weidong Xu (weidongxu-microsoft) merged 1 commit into
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 69da8151-cc69-4b91-9c23-92722f5777c0
Weidong Xu (weidongxu-microsoft)
requested review from
Mike Harder (mikeharder) and
Ray Chen (raych1)
as code owners
September 15, 2026 11:49
Copilot started reviewing on behalf of
Weidong Xu (weidongxu-microsoft)
September 15, 2026 11:50
View session
|
Azure Pipelines: Successfully started running 1 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. |
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The scoped path-length updates address the stated requirement with no unresolved review issues.
Pull request overview
Relaxes the Windows checkout path reservation from 38 to 30 characters for Java CI and patch-release pipelines.
Changes:
- Updates standard CI path validation.
- Applies the same limit to patch-release validation.
File summaries
| File | Description |
|---|---|
eng/pipelines/templates/stages/archetype-sdk-client-patch.yml |
Lowers the patch-release path reservation. |
eng/pipelines/templates/jobs/ci.yml |
Lowers the standard CI path reservation. |
Review details
- Files reviewed: 2/2 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.
Alan Zimmer (alzimmermsft)
approved these changes
Sep 15, 2026
Weidong Xu (weidongxu-microsoft)
merged commit Sep 15, 2026
9dd9bd0
into
Azure:main
22 checks passed
Weidong Xu (weidongxu-microsoft)
deleted the
weidongxu/relax-java-path-length-30
branch
September 15, 2026 12:38
pull Bot
pushed a commit
to jrcribb/cadl
that referenced
this pull request
Sep 16, 2026
## Summary - Align generated Java path calculations with the 30-character checkout-root reservation from Azure/azure-sdk-for-java#50462. - Preserve test and sample class names that now fit under the relaxed limit. - Apply the same reservation to GraalVM resource paths and update boundary tests. ## Validation - `npm run build` - `npm run format` - `npm run lint` - `pnpm format` - `mvn test --projects http-client-generator-core --define "test=ClassNameUtilTests" -f .\generator\pom.xml` --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 277f389f-e041-4191-8637-c3843c839b05
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This is the smallest practical relaxation needed by the Node Customization management SDK: its longest generated path is 229 repository-relative characters, totaling 259 with the new reservation.
Follow-up for #50458.