Skip to content

Conversation

pakrym-oai
Copy link
Collaborator

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

Comment on lines 81 to 84
const lines = ['Potential duplicates detected:'
...numbers.map((value) => `- #${value}`),
''
'*Powered by [Codex Action](https://github.com/openai/codex-action)*'];
Copy link
Contributor

Choose a reason for hiding this comment

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

P1 Badge Fix array literal syntax in duplicate comment construction

The array passed to lines now splits each element onto a new line but dropped the commas between them. As written (['Potential duplicates detected:' ...numbers.map(...), '' '*Powered …']) the Node runner will throw a SyntaxError before executing, so the workflow will fail and no comment will be posted. Add commas after the headline string and after the empty string so the array literal is valid.

Useful? React with 👍 / 👎.

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.

Neat!

@pakrym-oai pakrym-oai merged commit 2d6cd69 into main Oct 3, 2025
19 checks passed
@pakrym-oai pakrym-oai deleted the pakrym/enable-codex-workflows branch October 3, 2025 00:37
@github-actions github-actions bot locked and limited conversation to collaborators Oct 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants