-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Standardize template references to use absolute paths across .pipelines folder #26426
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: TravisEz13 <[email protected]>
|
@copilot this was supposed to standardize on use absolute paths |
…elines folder Co-authored-by: TravisEz13 <[email protected]>
Corrected. Standardized all template references across Updated 8 files to ensure consistency:
Commit: 40a470a |
There was a problem hiding this 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 standardizes template path references across the .pipelines folder to consistently use absolute paths with the format /.pipelines/templates/*.yml@self, improving clarity and maintainability of the Azure Pipeline configurations.
- Converts all relative template paths (
../,./, and bare filenames) to absolute paths - Ensures consistent
@selfsuffix for all template references - Affects 7 files across template and pipeline configuration files
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
.pipelines/templates/nupkg.yml |
Standardized 3 template references to use absolute paths |
.pipelines/templates/mac-package-build.yml |
Standardized 4 template references to use absolute paths |
.pipelines/templates/linux-package-build.yml |
Standardized 4 template references to use absolute paths |
.pipelines/templates/compliance/apiscan.yml |
Standardized 2 template references from relative ../ paths to absolute paths |
.pipelines/templates/SetVersionVariables.yml |
Standardized set-reporoot.yml reference to use absolute path |
.pipelines/PowerShell-vPack-Official.yml |
Standardized SetVersionVariables.yml reference from .pipelines/templates/ to absolute path with leading / |
.pipelines/MSIXBundle-vPack-Official.yml |
Standardized SetVersionVariables.yml reference from .pipelines/templates/ to absolute path with leading / |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
PR Summary
Standardizes all template path references across the
.pipelinesfolder to use absolute paths for consistency and clarity.PR Context
Template references in
.pipelineswere using inconsistent path formats - some used absolute paths (/.pipelines/templates/*.yml@self), others used relative paths (../../*.yml@self,../*.yml, or just filename). This PR standardizes all references to use absolute paths throughout.Updated 8 files to use consistent absolute path format
/.pipelines/templates/*.yml@self:Template files:
.pipelines/templates/packaging/windows/package.yml.pipelines/templates/mac-package-build.yml.pipelines/templates/linux-package-build.yml.pipelines/templates/nupkg.yml.pipelines/templates/SetVersionVariables.yml.pipelines/templates/compliance/apiscan.ymlPipeline files:
.pipelines/PowerShell-vPack-Official.yml.pipelines/MSIXBundle-vPack-Official.ymlTemplate references standardized:
SetVersionVariables.yml@self→/.pipelines/templates/SetVersionVariables.yml@selfshouldSign.yml→/.pipelines/templates/shouldSign.yml@selfcloneToOfficialPath.yml→/.pipelines/templates/cloneToOfficialPath.yml@selfrebuild-branch-check.yml@self→/.pipelines/templates/rebuild-branch-check.yml@selfset-reporoot.yml@self→/.pipelines/templates/set-reporoot.yml@selfinsert-nuget-config-azfeed.yml→/.pipelines/templates/insert-nuget-config-azfeed.yml@self../../*.yml@self→/.pipelines/templates/*.yml@self../*.yml→/.pipelines/templates/*.yml@self.pipelines/templates/*.yml@self→/.pipelines/templates/*.yml@selfAddresses feedback from #26425.
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright header💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.