Skip to content

Fix: Update README contributors workflow to create pull request instead of pushing directly #131

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 20, 2025

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented May 20, 2025

This PR modifies the update-readme-contributors.yml workflow to create a pull request instead of pushing changes directly to the main branch.

Changes:

  • Replaced manual git commands with the peter-evans/create-pull-request action
  • Added explicit permissions to the workflow to ensure it has the necessary access:
    • contents: write
    • pull-requests: write
  • Configured the PR creation with:
    • Descriptive title and body
    • Unique branch name with timestamp suffix
    • Automatic branch deletion after merge

Benefits:

  • Follows git best practices by not pushing directly to the main branch
  • Enables code review before changes are merged
  • Reduces the risk of unintended changes to the main branch
  • Maintains a clear history of contributor updates

The workflow continues to run on the same schedule and triggers, but now creates a PR instead of directly modifying the main branch.

Fixes #129.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@Copilot Copilot AI changed the title [WIP] Bug: ReadMe contributors workflow should create pull request instead of pushing to main branch Fix: Update README contributors workflow to create pull request instead of pushing directly May 20, 2025
@Copilot Copilot AI requested a review from deadlydog May 20, 2025 00:43
Copilot finished work on behalf of deadlydog May 20, 2025 00:43
Copy link

Overall Project 90.58% 🍏

There is no coverage information present for the Files changed

1 similar comment
Copy link

Overall Project 90.58% 🍏

There is no coverage information present for the Files changed

@deadlydog deadlydog marked this pull request as ready for review May 20, 2025 00:48
@deadlydog deadlydog merged commit d18141c into main May 20, 2025
2 checks passed
@deadlydog deadlydog deleted the copilot/fix-129-2 branch May 20, 2025 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: ReadMe contributors workflow should create pull request instead of pushing to main branch
2 participants