Skip to content

fix: update npm install display name to npm ci and adjust branch filter for pull requests#557

Merged
rido-min merged 2 commits intomainfrom
fix/release-gates
Sep 8, 2025
Merged

fix: update npm install display name to npm ci and adjust branch filter for pull requests#557
rido-min merged 2 commits intomainfrom
fix/release-gates

Conversation

@rido-min
Copy link
Copy Markdown
Member

@rido-min rido-min commented Sep 3, 2025

No description provided.

Copilot AI review requested due to automatic review settings September 3, 2025 23:28
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates CI/CD configurations to improve build reliability and branch management. Changes the npm install step to use npm ci and adjusts branch filtering for pull requests.

  • Replace npm install with npm ci for deterministic dependency installation
  • Update pull request trigger to target release branches instead of dev branches

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/ci.yml Updates branch filter to trigger on release/* instead of dev* branches
.azdo/ci-pr.yaml Changes display name from 'npm install' to 'npm ci' for consistency

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .azdo/ci-pr.yaml
@rido-min rido-min enabled auto-merge (squash) September 8, 2025 19:16
@rido-min rido-min merged commit 8bbf355 into main Sep 8, 2025
7 checks passed
@rido-min rido-min deleted the fix/release-gates branch September 8, 2025 20:35
rido-min added a commit that referenced this pull request Sep 8, 2025
…er for pull requests (#557)

* fix: update npm install display name to npm ci and adjust branch filter for pull requests

* fix: update branch filter in CI configuration to include release branches

---------

Co-authored-by: Rido <nouser@gh.com>
rido-min added a commit that referenced this pull request Sep 8, 2025
* Reorder fileDownloaders execution in agentApplication (#543)

* chore: update dependencies in package.json and package-lock.json (#549)

Co-authored-by: Rido <nouser@gh.com>

* fix: update npm install display name to npm ci and adjust branch filter for pull requests (#557)

* fix: update npm install display name to npm ci and adjust branch filter for pull requests

* fix: update branch filter in CI configuration to include release branches

---------

Co-authored-by: Rido <nouser@gh.com>

* Add citation URL to streaming response metadata (#558)

* citation url

* add streamingResponse tests

---------

Co-authored-by: Rido <nouser@gh.com>

* Restore teams-extensions (#548)

* restore teams-extensions. Pending Tests verification

* fixes @microsoft/agents-hosting-extensions-teams TeamsInfo.getTeamChannels wrongly typed
Fixes #507

* fix: correct typos in tenantId parameter and test descriptions

---------

Co-authored-by: Rido <nouser@gh.com>

* fix lock

* rolback deps

---------

Co-authored-by: Cecilia Avila <44245136+ceciliaavila@users.noreply.github.com>
Co-authored-by: Rido <nouser@gh.com>
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