Skip to content

Added github workflows. - #4

Merged
Mike Krüger (mkrueger) merged 30 commits into
mainfrom
dev/mkrueger/workflows
Apr 17, 2026
Merged

Added github workflows.#4
Mike Krüger (mkrueger) merged 30 commits into
mainfrom
dev/mkrueger/workflows

Conversation

@mkrueger

@mkrueger Mike Krüger (mkrueger) commented Mar 18, 2026

Copy link
Copy Markdown
Collaborator

Adds GitHub Actions validation and packaging workflow support for the repo.

This PR also updates CI/release infrastructure around that migration:

  • removes the old Azure Pipelines pull request workflow
  • updates the official Azure Pipelines packaging/versioning flow for preview signed packages
  • refreshes README and CONTRIBUTING guidance for the current GitHub Actions and Azure Pipelines setup

@mkrueger

Copy link
Copy Markdown
Collaborator Author

Issue is that I can't change the variables in the azure pipeline I need to set the GitHubReleaseToken there.

Copilot AI review requested due to automatic review settings March 18, 2026 13:03
@mkrueger
Mike Krüger (mkrueger) requested a review from a team March 18, 2026 13:03

This comment was marked as resolved.

Copilot AI review requested due to automatic review settings March 19, 2026 09:00

This comment was marked as resolved.

Copilot AI review requested due to automatic review settings March 19, 2026 10:30

This comment was marked as resolved.

Copilot AI review requested due to automatic review settings March 19, 2026 13:31

This comment was marked as resolved.

This comment was marked as resolved.

Copilot AI review requested due to automatic review settings March 20, 2026 11:45
@mkrueger

Copy link
Copy Markdown
Collaborator Author

Copilot open a new pull request to apply changes based on the comments in this thread

Copilot AI commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

Mike Krüger (@mkrueger) I've opened a new pull request, #5, to work on those changes. Once the pull request is ready, I'll request review from you.

This comment was marked as resolved.

Copilot AI review requested due to automatic review settings April 16, 2026 13:43

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

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Comment thread CosmosDBShell/Azure.Data.Cosmos.Shell.Core/ShellInterpreter.cs
Comment thread .github/workflows/validate-and-package.yml
Copilot AI review requested due to automatic review settings April 16, 2026 14:00

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

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.

Comment thread .pipelines/CosmosDB-Shell-Official.yml Outdated
Comment thread .pipelines/CosmosDB-Shell-Official.yml Outdated
Comment thread .github/workflows/validate-and-package.yml
Copilot AI review requested due to automatic review settings April 16, 2026 14:44

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

Copilot reviewed 9 out of 9 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings April 16, 2026 14:57

This comment was marked as outdated.

The pack step was failing with NETSDK1047 because it ran with --no-build
--no-restore, but the project.assets.json only had the last RID from the
publish loop. Tool packaging with ToolPackageRuntimeIdentifiers needs
assets for ALL RIDs.

Fix:
- Add restore without -r flag before pack (restores all RIDs)
- Remove --no-build so pack can build each RID package
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.

4 participants