-
Notifications
You must be signed in to change notification settings - Fork 42
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
Enh deploy release v6 #278
Conversation
@@ -127,11 +126,12 @@ function Pack($envName, $environment, $workingDirectory) { | |||
|
|||
& ./node_modules/.bin/tfx extension create --root $workingDirectory --manifest-globs extension-manifest.json --overridesFile $overridesFile --outputPath "$buildArtifactsPath/$environment" --no-prompt | |||
|
|||
if (-not $?) {throw "Failed to create extension. Exit Code $LASTEXITCODE"} | |||
& npm uninstall tfx-cli |
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.
This is the only change to this line, the rest was auto-formatting kicking in
import { getInputParameters } from "./input-parameters"; | ||
import * as tasks from "azure-pipelines-task-lib/task"; | ||
|
||
export class MockTaskWrapper implements TaskWrapper { |
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.
This needs to move to a common location, but I initially had a reference issue so I put it here for now
"label": "Additional Arguments (deprecated)", | ||
"defaultValue": "", | ||
"required": false, | ||
"helpMarkDown": "Additional arguments are no longer supported, move these values to the appropriate fields", |
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.
Wording?
], | ||
"instanceNameFormat": "Deploy Octopus Release", | ||
"execution": { | ||
"Node16": { |
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.
updated
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.
👍
import commandLineArgs from "command-line-args"; | ||
import shlex from "shlex"; | ||
|
||
describe("cli args", () => { |
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.
These tests were me working out how to parse the arguments from the AdditionalArguments
field, thought I would leave them here for future reference
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.
Looking good. Offered some ideas on the wording for the no longer supported fields. vsts.md
hasn't been updated.
Co-authored-by: Shannon Lewis <[email protected]>
Co-authored-by: Shannon Lewis <[email protected]>
feat: New steps for Zip/NuGet pack (#276) feat:push step v6 (#277) feat: deploy release v6 (#278) feat: deploy tenant release v6 (#281) feat: await task (#282) feat: Run runbook step (#285) feat: Create release v6 (#283) chore: updated the way user agent strings are built for the v6 steps (#287) feat: push build information v6 (#284) chore: Added output variable definitions for the Pack steps feat: CLI installer step - v6 (#279) chore: added warning logs for use of additional arguments fields (#286) chore: fixed some pluralisation inconsistencies Co-authored-by: Ben Pearce <[email protected]>
Supports variables coming from either additional arguments or the variable field
In ui mode