Skip to content

Add CI workflow and NuGet publishing automation#1

Merged
xqiu merged 1 commit intomainfrom
codex/create-nuget-package-for-python-integration
Nov 8, 2025
Merged

Add CI workflow and NuGet publishing automation#1
xqiu merged 1 commit intomainfrom
codex/create-nuget-package-for-python-integration

Conversation

@xqiu
Copy link
Owner

@xqiu xqiu commented Nov 8, 2025

Summary

  • add a GitHub Actions CI workflow that restores, builds, tests, and packs the solution on pushes and pull requests
  • introduce a workflow for manually publishing the NuGet package to NuGet.org using a stored API key
  • provide cross-platform scripts and a solution file to support packing and pushing the package locally

Testing

  • not run (dotnet CLI unavailable in container)

Codex Task

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@@ -0,0 +1 @@
print("get-pip placeholder")

Choose a reason for hiding this comment

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

P1 Badge Ship real get-pip bootstrapper

The Init flow copies get-pip.py next to the library, runs it to install pip, and then creates the virtual environment. The committed get-pip.py is only print("get-pip placeholder"), so executing it never installs pip. As a result the new virtual environment will be created without pip and InstallRequirement will reliably fail (python -m pip install exits non‑zero) whenever dependencies are installed. This file needs to contain the actual get‑pip bootstrap script or the code should use another supported mechanism such as python -m ensurepip.

Useful? React with 👍 / 👎.

@xqiu xqiu merged commit 5ab2613 into main Nov 8, 2025
1 check failed
@xqiu xqiu deleted the codex/create-nuget-package-for-python-integration branch November 8, 2025 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant