-
Notifications
You must be signed in to change notification settings - Fork 221
Update APIView script path to absolute path #8283
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
Update APIView script path to absolute path #8283
Conversation
|
The following pipelines have been queued for testing: |
| - ${{ if or(ne(parameters.GenerateApiReviewForManualOnly, true), eq(variables['Build.Reason'], 'Manual')) }}: | ||
| - task: Powershell@2 | ||
| inputs: | ||
| filePath: ./eng/common/scripts/Create-APIReview.ps1 |
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.
Why didn't this work?
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.
PowerApps task powershell@2 did not honor the WorkingDirectory param. I see that it's documented but current directory was still default source path (first cloned repo). I will check more to find out the root cause with a small repro and report an issue in Devops page if this turns out as an issue in the task itself.
weshaggard
left a comment
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.
curious why the other option didn't work but fine with this as well.
Use absolute path with parameter root path as prefix to run create api review script so that this template can be used in Java release pipeline.