Compare commits
15 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
bd57348f7a | ||
|
|
cd08d4e74c | ||
|
|
29dbfc0acd | ||
|
|
c0bf76830b | ||
|
|
a08a03a717 | ||
|
|
1cb1956a68 | ||
|
|
6e7a9cb309 | ||
|
|
2ca266185d | ||
|
|
e2a437cdbd | ||
|
|
2d5c80f215 | ||
|
|
fc56ed279d | ||
|
|
7f0d383d15 | ||
|
|
89a865123c | ||
|
|
260ef70e70 | ||
|
|
2293747d8b |
@@ -14,6 +14,8 @@ represented by the pull requests that fixed them. Critical items to know are:
|
||||
Versions correspond with GitHub releases that can be referenced with @ using actions.
|
||||
|
||||
## [master](https://github.com/vsoch/pull-request-action/tree/master) (master)
|
||||
- bugfix of missing from branch with scheduled run (1.0.16)
|
||||
- forgot to add assignees (1.0.15)
|
||||
- output and environment variables for PR number and return codes (1.0.5)
|
||||
- added support for reviewer (individual and team) assignments (1.0.4)
|
||||
- added support for maintainer can modify and assignees (1.0.3)
|
||||
|
||||
11
README.md
11
README.md
@@ -22,13 +22,16 @@ jobs:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: pull-request-action
|
||||
uses: vsoch/pull-request-action@1.0.6
|
||||
uses: vsoch/pull-request-action@master
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
BRANCH_PREFIX: "update/"
|
||||
PULL_REQUEST_BRANCH: "master"
|
||||
```
|
||||
|
||||
**Important**: Make sure to use a stable [release](https://github.com/vsoch/pull-request-action/releases) instead of a branch for your workflow.
|
||||
|
||||
|
||||
## Environment Variable Inputs
|
||||
|
||||
Unlike standard actions, this action just uses variables from the environment.
|
||||
@@ -36,6 +39,8 @@ Unlike standard actions, this action just uses variables from the environment.
|
||||
| Name | Description | Required | Default |
|
||||
|------|-------------|----------|---------|
|
||||
| BRANCH_PREFIX | the prefix to filter to. If the branch doesn't start with the prefix, it will be ignored | false | "" |
|
||||
| PULL_REQUEST_REPOSITORY | Choose another repository instead of default GITHUB_REPOSITORY for the PR | false | |
|
||||
| PULL_REQUEST_TOKEN | [Personal Access Token(PAT)](https://docs.github.com/en/github/authenticating-to-github/creating-a-personal-access-token) only if you define a different repository with PULL_REQUEST_REPOSITORY | false | |
|
||||
| PULL_REQUEST_BRANCH | open pull request against this branch | false | master |
|
||||
| PULL_REQUEST_FROM_BRANCH | if a branch isn't found in your GitHub payload, use this branch | false | |
|
||||
| PULL_REQUEST_BODY | the body for the pull request | false | |
|
||||
@@ -68,6 +73,8 @@ an issue or PR, they are ignored otherwise.
|
||||
The `GITHUB_TOKEN` secret is required to interact and authenticate with the GitHub API to open
|
||||
the pull request. The example is [deployed here](https://github.com/vsoch/pull-request-action-example) with an example opened (and merged) [pull request here](https://github.com/vsoch/pull-request-action-example/pull/1) if needed.
|
||||
|
||||
If you want to create a pull request to another repository, for example, a pull request to the upstream repository, you need to define PULL_REQUEST_REPOSITORY and PULL_REQUEST_TOKEN. The PULL_REQUEST_TOKEN is one [Personal Access Token(PAT)](https://docs.github.com/en/github/authenticating-to-github/creating-a-personal-access-token), which can be save in the [encrypted secrets](https://docs.github.com/en/actions/reference/encrypted-secrets#creating-encrypted-secrets-for-a-repository)
|
||||
|
||||
## Outputs
|
||||
|
||||
The action sets a few useful output and environment variables. An output can
|
||||
@@ -114,7 +121,7 @@ jobs:
|
||||
PR_BRANCH_FROM=release-v$(cat VERSION)
|
||||
export "PULL_REQUEST_FROM_BRANCH=${PR_BRANCH_FROM}" >> $GITHUB_ENV
|
||||
- name: pull-request-action
|
||||
uses: vsoch/pull-request-action@1.0.6
|
||||
uses: vsoch/pull-request-action@master
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
PULL_REQUEST_BRANCH: "master"
|
||||
|
||||
@@ -9,7 +9,7 @@ jobs:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: pull-request-action
|
||||
uses: vsoch/pull-request-action@1.0.3
|
||||
uses: vsoch/pull-request-action@master
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
BRANCH_PREFIX: "update/"
|
||||
|
||||
@@ -43,7 +43,7 @@ jobs:
|
||||
echo "PULL_REQUEST_FROM_BRANCH=${BRANCH_FROM}" >> $GITHUB_ENV
|
||||
|
||||
- name: Open Pull Request
|
||||
uses: vsoch/pull-request-action@1.0.6
|
||||
uses: vsoch/pull-request-action@master
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
PULL_REQUEST_BRANCH: "master"
|
||||
|
||||
@@ -47,7 +47,7 @@ jobs:
|
||||
- name: Set PR body
|
||||
run: echo "PULL_REQUEST_BODY=${{steps.set-pr-body.outputs.result}}" >> $GITHUB_ENV
|
||||
- name: pull-request-action
|
||||
uses: vsoch/pull-request-action@1.0.10
|
||||
uses: vsoch/pull-request-action@master
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
BRANCH_PREFIX: "hotfix-"
|
||||
|
||||
@@ -10,7 +10,7 @@ jobs:
|
||||
steps:
|
||||
- name: pull-request-action
|
||||
id: pull_request
|
||||
uses: vsoch/pull-request-action@1.0.6
|
||||
uses: vsoch/pull-request-action@master
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
BRANCH_PREFIX: "update/"
|
||||
|
||||
@@ -11,7 +11,7 @@ jobs:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: pull-request-action
|
||||
uses: vsoch/pull-request-action@add/support-null-branch
|
||||
uses: vsoch/pull-request-action@master
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
BRANCH_PREFIX: "update/"
|
||||
|
||||
@@ -39,7 +39,7 @@ jobs:
|
||||
git push --force "https://${{ github.actor }}:${{secrets.GITHUB_TOKEN}}@github.com/${{ github.repository }}.git" "HEAD:${{ steps.data.outputs.suite_update_branch }}"
|
||||
|
||||
- name: Open a PR to the default branch
|
||||
uses: vsoch/pull-request-action@1.0.2
|
||||
uses: vsoch/pull-request-action@master
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
PULL_REQUEST_FROM_BRANCH: "${{ steps.data.outputs.suite_update_branch }}"
|
||||
|
||||
@@ -9,7 +9,7 @@ jobs:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: pull-request-action
|
||||
uses: vsoch/pull-request-action@1.0.4
|
||||
uses: vsoch/pull-request-action@master
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
BRANCH_PREFIX: "update/"
|
||||
|
||||
@@ -108,6 +108,7 @@ def open_pull_request(title, body, target, source, is_draft=False, can_modify=Tr
|
||||
print("Data for opening pull request: %s" % data)
|
||||
response = requests.post(PULLS_URL, json=data, headers=HEADERS)
|
||||
if response.status_code != 201:
|
||||
print(f"pull request url is {PULLS_URL}")
|
||||
abort_if_fail(response, "Unable to create pull request")
|
||||
|
||||
return response
|
||||
@@ -188,11 +189,12 @@ def list_pull_requests(target, source):
|
||||
return response.json()
|
||||
|
||||
|
||||
def add_assignees(entry):
|
||||
def add_assignees(entry, assignees):
|
||||
"""Given a pull request metadata (from create or update) add assignees
|
||||
|
||||
Parameters:
|
||||
entry (dict) : the pull request metadata
|
||||
entry (dict) : the pull request metadata
|
||||
assignees (str) : comma separated assignees string set by action
|
||||
"""
|
||||
# Remove leading and trailing quotes
|
||||
assignees = parse_into_list(assignees)
|
||||
@@ -232,6 +234,22 @@ def find_pull_request(listing, source):
|
||||
return entry
|
||||
|
||||
|
||||
def find_default_branch():
|
||||
"""Find default branch for a repo (only called if branch not provided)
|
||||
"""
|
||||
response = requests.get(REPO_URL)
|
||||
|
||||
# Case 1: 404 might need a token
|
||||
if response.status_code == 404:
|
||||
response = requests.get(REPO_URL, headers=HEADERS)
|
||||
if response.status_code != 200:
|
||||
abort_if_fail(response, "Unable to retrieve default branch")
|
||||
|
||||
default_branch = response.json()["default_branch"]
|
||||
print("Found default branch: %s" % default_branch)
|
||||
return default_branch
|
||||
|
||||
|
||||
def add_reviewers(entry, reviewers, team_reviewers):
|
||||
"""Given regular or team reviewers, add them to a PR.
|
||||
|
||||
@@ -267,14 +285,17 @@ def add_reviewers(entry, reviewers, team_reviewers):
|
||||
API_VERSION = "v3"
|
||||
BASE = "https://api.github.com"
|
||||
|
||||
PR_TOKEN = os.environ.get("PULL_REQUEST_TOKEN") or get_envar("GITHUB_TOKEN")
|
||||
PR_REPO = os.environ.get("PULL_REQUEST_REPOSITORY") or get_envar("GITHUB_REPOSITORY")
|
||||
|
||||
HEADERS = {
|
||||
"Authorization": "token %s" % get_envar("GITHUB_TOKEN"),
|
||||
"Authorization": "token %s" % PR_TOKEN,
|
||||
"Accept": "application/vnd.github.%s+json;application/vnd.github.antiope-preview+json;application/vnd.github.shadow-cat-preview+json"
|
||||
% API_VERSION,
|
||||
}
|
||||
|
||||
# URLs
|
||||
REPO_URL = "%s/repos/%s" % (BASE, get_envar("GITHUB_REPOSITORY"))
|
||||
REPO_URL = "%s/repos/%s" % (BASE, PR_REPO)
|
||||
ISSUE_URL = "%s/issues" % REPO_URL
|
||||
PULLS_URL = "%s/pulls" % REPO_URL
|
||||
|
||||
@@ -320,7 +341,7 @@ def create_pull_request(
|
||||
|
||||
# If we have opened or updated, we can add assignees
|
||||
if response and assignees:
|
||||
add_assignees(response)
|
||||
add_assignees(response, assignees)
|
||||
if response and (reviewers or team_reviewers):
|
||||
add_reviewers(response, reviewers, team_reviewers)
|
||||
|
||||
@@ -338,8 +359,8 @@ def main():
|
||||
if not branch_prefix:
|
||||
print("No branch prefix is set, all branches will be used.")
|
||||
|
||||
# Default to master to support older, will eventually change to main
|
||||
pull_request_branch = os.environ.get("PULL_REQUEST_BRANCH", "master")
|
||||
# Default to project default branch if none provided
|
||||
pull_request_branch = os.environ.get("PULL_REQUEST_BRANCH", find_default_branch())
|
||||
print("Pull requests will go to %s" % pull_request_branch)
|
||||
|
||||
# Pull request draft
|
||||
@@ -393,7 +414,7 @@ def main():
|
||||
if not from_branch:
|
||||
print("PULL_REQUEST_FROM_BRANCH is not set, checking branch in payload.")
|
||||
with open(check_events_json(), "r") as fd:
|
||||
from_branch = json.loads(fd.read()).get("ref")
|
||||
from_branch = json.loads(fd.read()).get("ref", "")
|
||||
from_branch = from_branch.replace("refs/heads/", "").strip("/")
|
||||
else:
|
||||
print("PULL_REQUEST_FROM_BRANCH is set.")
|
||||
|
||||
Reference in New Issue
Block a user