- Notifications
You must be signed in to change notification settings - Fork1.2k
Don't actually create a PR#589
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
👋 Hi there! It looks like this PR is targeting the If this is a bug fix that requires a patch release __ (e.g., a critical bug that needs to be fixed before the next release), please leave the base branch as You can change this by:
Thanks for your contribution! 🚀 |
3deaca8 intomainUh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This PR removes automated pull request creation for branch sync and release, replacing it with manual instructions to comply with organizational policies.
- Removed the automated PR creation logic in the branch sync workflow and replaced it with manual sync instructions.
- Replaced the automated release PR creation in the release workflow with tagging and manual PR creation instructions.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| .github/workflows/sync-next-branch.yml | Replaces automated sync PR creation with manual instructions. |
| .github/workflows/release.yml | Removes automated PR creation for releases and updates release steps. |
| GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| echo "action-needed=true" >> $GITHUB_OUTPUT | ||
| echo "🔄 Next branch needs syncing (${BEHIND_COUNT} commits behind main)" | ||
| echo "� Manual PR creation required due to organization policies" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
The '�' character appears to be a mis-encoded or unintended symbol. Replace it with a proper emoji or text to improve clarity.
| echo "� Manual PR creation required due to organization policies" | |
| echo "⚠️ Manual PR creation required due to organization policies" |
Copilot uses AI. Check for mistakes.
| echo "5. 🔀 Merge the pull request into main: ${PR_URL}" | ||
| echo "6. Post message in #gh-mcp-releases channel in Slack and then share to the other mcp channels" | ||
| echo "5. � Create a Pull Request from 'next' to 'main' branch with title 'Release $TAG'" | ||
| echo "6. �🔀 Merge the pull request into main" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
The '�' symbol in the instructions seems to be an encoding error. Consider replacing it with a clear and appropriate symbol or text.
| echo "6. �🔀 Merge the pull request intomain" | |
| echo "5. 🔄 Create a Pull Request from 'next' to 'main' branch with title 'Release $TAG'" | |
| echo "6. 🔀 Merge the pull request into main" |
Copilot uses AI. Check for mistakes.
Closes: