Skip to content

Fix: Change file name to start with small letter instead of captial l…#356

Merged
tibo-openai merged 3 commits intoopenai:mainfrom
AbdelrhmanX7:abdelrhman/fix/change-file-name-to-start-with-small-letter-instead-of-captial
Apr 18, 2025
Merged

Fix: Change file name to start with small letter instead of captial l…#356
tibo-openai merged 3 commits intoopenai:mainfrom
AbdelrhmanX7:abdelrhman/fix/change-file-name-to-start-with-small-letter-instead-of-captial

Conversation

@AbdelrhmanX7
Copy link
Copy Markdown
Contributor

No description provided.

@tibo-openai tibo-openai merged commit d69a17a into openai:main Apr 18, 2025
2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Apr 18, 2025
@tibo-openai
Copy link
Copy Markdown
Collaborator

ty, and keep contributing!

zemaj added a commit to zemaj/codex that referenced this pull request Oct 28, 2025
The preview-build workflow was attempting to publish prereleases for all
PRs, including those from forks. Fork PRs don't have write access to
create releases, causing the publish job to fail.

This change adds a condition to only run the release and comment jobs for
PRs originating from the main repository (not forks) by checking:
  github.event.pull_request.head.repo.full_name == github.repository

The build job will still run for all PRs (including forks) to validate
the code, but publishing is now limited to trusted contexts.

Fixes: openai#355
Fixes: openai#356

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
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