Skip to content

fix: use artifactName for pipelineArtifact inputs in publish pipeline#625

Merged
corinagum merged 1 commit into
mainfrom
cg/fix-publish-inputs
Jun 17, 2026
Merged

fix: use artifactName for pipelineArtifact inputs in publish pipeline#625
corinagum merged 1 commit into
mainfrom
cg/fix-publish-inputs

Conversation

@corinagum

Copy link
Copy Markdown
Collaborator

The 1ES template requires artifactName (not artifact) for pipelineArtifact input declarations in templateContext. This was causing:

Unexpected value 'artifactName is a required argument for pipelineArtifact input and 1ES.DownloadPipelineArtifact@1 task'

The 1ES template requires 'artifactName' (not 'artifact') for
pipelineArtifact input declarations in templateContext.

Co-authored-by: Copilot <[email protected]>
Copilot AI review requested due to automatic review settings June 17, 2026 22:48

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.

Pull request overview

Updates the Azure DevOps publish pipeline to align with the 1ES template schema by using the required artifactName key for pipelineArtifact inputs, resolving the reported template validation error during artifact download.

Changes:

  • Replaced artifact with artifactName in templateContext.inputs for pipelineArtifact declarations in the publish stage jobs.
  • Ensured artifact input declarations match the already-used artifactName pattern in templateContext.outputs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@corinagum
corinagum added this pull request to the merge queue Jun 17, 2026
corinagum added a commit to microsoft/teams-sdk that referenced this pull request Jun 17, 2026
…ation (#2894)

Same fix as microsoft/teams.ts#625 and microsoft/teams.py#481.

Splits the single-job publish pipeline into a separate `publish` stage
with two jobs:
- `publish_internal` - no release context, publishes to Azure Artifacts
- `publish_public` - `type: releaseJob` + `isProduction: true`,
publishes via ESRP

This fixes the 1ES template error:
```
Change management: no release tasks outside of release jobs in pipeline
```

Also uses `artifactName` (required by 1ES template) for
`pipelineArtifact` inputs.

Co-authored-by: Copilot <[email protected]>
Merged via the queue into main with commit d592187 Jun 17, 2026
7 checks passed
@corinagum
corinagum deleted the cg/fix-publish-inputs branch June 17, 2026 23:01
@lilyydu lilyydu mentioned this pull request Jul 16, 2026
lilyydu added a commit that referenced this pull request Jul 16, 2026
## Release v2.0.14

Prepares the stable **2.0.14** release by merging `main` into `release`
and setting `version.json` to the stable version (removing the
`-preview.{height}` suffix).

- Previous release: `2.0.13` (published 2026-06-15)
- Only functional diff vs `main` is `version.json`:
`2.0.14-preview.{height}` → `2.0.14`

## What's in this release

### 🚀 Features
* Flatten client method chains to reduce call hops by @lilyydu in
#634
* Support resetting response streams and streaming error handling by
@lilyydu in #635
* Add `extendedMarkdown` text format value by @singhk97 in
#615

### 🐛 Fixes
* Set `replyToId` for all streaming activities by @lilyydu in
#646
* Use `artifactName` for pipelineArtifact inputs in publish pipeline by
@corinagum in #625

### ♻️ Refactors
* Move `apps` from file-based mixins to real classes by @heyitsaamir in
#633
* Split up Outbound and Inbound Activities by @heyitsaamir in
#639
* Remove `ExperimentalTeamsQuotedReplies` markers by @corinagum in
#621

### 🧪 Tests
* Add integration tests for Teams TypeScript SDK by @corinagum in
#622

### 📚 Docs & Examples
* Migrate examples to the flattened conversation API by @lilyydu in
#637
* Note npm CFS proxy for Microsoft-managed devices by @corinagum in
#638
* Update integration test runbook link to ADO wiki by @corinagum in
#624

### 🔧 Chores & Dependencies
* Bump version to 2.0.14-preview and fix pipeline release job by
@corinagum in #620
* Bump hono from 4.12.21 to 4.12.25 by @dependabot in
#623
* Bump react-router from 7.15.0 to 7.15.1 by @dependabot in
#619
* Bump vite from 6.4.2 to 6.4.3 by @dependabot in
#617

**Full Changelog**:
v2.0.13...prep-release/2.0.14

### After merge
1. Trigger the [release
pipeline](https://dev.azure.com/DomoreexpGithub/Github_Pipelines/_build?definitionId=52&_a=summary)
on `release` with **Public** publish type
2. Bump `main` → `2.0.15-preview.{height}`
3. Create the `v2.0.14` git tag + GitHub Release

Co-authored-by: Copilot <[email protected]>
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