Skip to content

Added github workflows.#4

Merged
mkrueger merged 30 commits into
mainfrom
dev/mkrueger/workflows
Apr 17, 2026
Merged

Added github workflows.#4
mkrueger merged 30 commits into
mainfrom
dev/mkrueger/workflows

Conversation

@mkrueger
Copy link
Copy Markdown
Contributor

@mkrueger mkrueger commented Mar 18, 2026

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 mkrueger requested a review from sevoku March 18, 2026 11:06
@mkrueger
Copy link
Copy Markdown
Contributor 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 mkrueger force-pushed the dev/mkrueger/workflows branch from 4731db3 to f8f40d6 Compare March 18, 2026 13:03
@mkrueger mkrueger requested a review from a team March 18, 2026 13:03

This comment was marked as resolved.

@mkrueger mkrueger force-pushed the dev/mkrueger/workflows branch from d5091ba to 75a2cae Compare March 19, 2026 09:00
Copilot AI review requested due to automatic review settings March 19, 2026 09:00

This comment was marked as resolved.

@mkrueger mkrueger force-pushed the dev/mkrueger/workflows branch from 75a2cae to 80b4b97 Compare March 19, 2026 09:52
Copilot AI review requested due to automatic review settings March 19, 2026 10:30
@mkrueger mkrueger force-pushed the dev/mkrueger/workflows branch from 80b4b97 to 181c9b7 Compare March 19, 2026 10:30

This comment was marked as resolved.

@mkrueger mkrueger force-pushed the dev/mkrueger/workflows branch from 181c9b7 to 0a59649 Compare March 19, 2026 13:01
Copilot AI review requested due to automatic review settings March 19, 2026 13:31
@mkrueger mkrueger force-pushed the dev/mkrueger/workflows branch from 0a59649 to 7992486 Compare March 19, 2026 13:31

This comment was marked as resolved.

@mkrueger mkrueger force-pushed the dev/mkrueger/workflows branch from 7992486 to 3619cd5 Compare March 19, 2026 14:34
Copilot AI review requested due to automatic review settings March 20, 2026 11:05
@mkrueger mkrueger force-pushed the dev/mkrueger/workflows branch from 3619cd5 to bf2723b Compare March 20, 2026 11:05

This comment was marked as resolved.

@mkrueger mkrueger force-pushed the dev/mkrueger/workflows branch from bf2723b to 1b4c410 Compare March 20, 2026 11:44
Copilot AI review requested due to automatic review settings March 20, 2026 11:45
@mkrueger
Copy link
Copy Markdown
Contributor Author

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

Copy link
Copy Markdown

Copilot AI commented Mar 20, 2026

@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
@mkrueger mkrueger force-pushed the dev/mkrueger/workflows branch from 9ea89b5 to 0adb738 Compare April 16, 2026 13:43
Copy link
Copy Markdown

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

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
Copy link
Copy Markdown

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

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
Copy link
Copy Markdown

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

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
@mkrueger mkrueger force-pushed the dev/mkrueger/workflows branch from ae521d7 to 85dc461 Compare April 16, 2026 15:13
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