-
Notifications
You must be signed in to change notification settings - Fork 2
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
NEW Create GitHub Action for adding PRs to the new project #1
NEW Create GitHub Action for adding PRs to the new project #1
Conversation
684885a
to
162c825
Compare
action.yml
Outdated
app-id: ${{ vars.PROJECT_MANAGER_APP_ID }} | ||
private-key: ${{ secrets.PROJECT_MANAGER_PRIVATE_KEY }} |
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.
These are stored in the silverstripe org.
162c825
to
90729c3
Compare
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.
You should add in the workflow files that would eventually get added by module-standardiser, in particular auto-tag.yml
There is a PR to add this repo to standardiser. I'll run standardiser against it after all PRs are merged (I have to run it anyway to get the new action on all relevant repos) |
90729c3
to
7369730
Compare
README.md
Outdated
|
||
**.github/workflows/add-pr-to-project.yml** | ||
```yml | ||
name: Add pull request to project |
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.
name: Add pull request to project | |
name: Add new pull requests to a github project |
Get it to match module-standardiser pr
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.
Done
README.md
Outdated
if: github.repository_owner == 'silverstripe' | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Add PR to project |
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.
- name: Add PR to project | |
- name: Add PR to github project |
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.
Done
7369730
to
caff90c
Compare
This action follows the docs for using GitHub Actions to add PRs to a project.
There is a GitHub App which holds read-only access for issues and PRs, and write access to projects. This action uses that app to generate a token, then uses the official GitHub Action for adding PRs to the project.
We are explicitly ignoring PRs created by the CMS Squad, because the project board is for community contributions. We don't want to waste community time reviewing and merging the CMS Squad PRs when we're already being paid to do that ourselves.
Issue