Skip to content

Conversation

@iceweasel-oai
Copy link
Contributor

No description provided.

Copy link
Contributor

@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".

@etraut-openai etraut-openai added the oai-pr PRs posted by Codex team members label Nov 11, 2025
Copy link
Collaborator

@bolinfest bolinfest left a comment

Choose a reason for hiding this comment

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

Thanks for doing this!

shell: bash
continue-on-error: true
run: |
set -euo pipefail
Copy link
Collaborator

Choose a reason for hiding this comment

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

FYI, I think GitHub sets this for shell: bash.

-X PATCH \
-f sha="${GITHUB_SHA}" \
-F force=true
-F force=true
Copy link
Collaborator

Choose a reason for hiding this comment

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

undo this un-indent?

@@ -0,0 +1,36 @@
WinGet manifests for the Codex CLI
Copy link
Collaborator

Choose a reason for hiding this comment

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

How about we make this .github/workflows/winget.md instead?

id: win_hash
shell: bash
run: |
set -euo pipefail
Copy link
Collaborator

Choose a reason for hiding this comment

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

this one too, btw

@iceweasel-oai
Copy link
Contributor Author

gonna land this since @bolinfest is on PTO. Will follow-up with some fixes from the comments.

@iceweasel-oai iceweasel-oai merged commit 37fba28 into main Nov 14, 2025
25 checks passed
@iceweasel-oai iceweasel-oai deleted the dev/iceweasel/winget-manifest branch November 14, 2025 19:06
@github-actions github-actions bot locked and limited conversation to collaborators Nov 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

oai-pr PRs posted by Codex team members

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants