Skip to content

Commit

Permalink
Merge pull request #58 from iamsobanjaved/iamsobanjaved/fix-auto-merg…
Browse files Browse the repository at this point in the history
…e-workflow

fix: use correct command for merging
  • Loading branch information
Feanil Patel authored Feb 16, 2023
2 parents e92170f + c6849d9 commit 34c0000
Showing 1 changed file with 10 additions and 3 deletions.
13 changes: 10 additions & 3 deletions .github/workflows/merge-python-requirements-upgrade-prs.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,17 @@
# This file contains a workflow which will be run on schedule every day, and
# will collect all the eligible (PRs which are labelled "Ready to merge" by our
# script) python upgrade PRs from both edx and openedx orgs.
# After fetching all PRs, it will check if all the checks passing on the PR, if yes
# then it will merge that PR, otherwise it will fail.

name: Merge Python Requirements Upgrade PRs

on:
schedule:
# will run the job at 10 AM every day (UTC),
# https://crontab.guru/#0_10_*_*_*
- cron: "0 10 * * *"
workflow_dispatch:

jobs:
get_list_of_prs:
Expand Down Expand Up @@ -80,11 +87,11 @@ jobs:
env:
GIT_TOKEN: ${{ secrets.requirements_bot_github_token }}
run: |
check_pr_tests_status.py --org "$org_name" --repo "$repo_name" --pr_number "$pr_number" \
--exclude-contexts 'Renovate|[Cc]odecov|Dependabot|edx-platform-ci' --all-checks
check_pr_tests_status.py --org "${{ env.org_name }}" --repo "${{ env.repo_name }}" --pr_number \
"${{ env.pr_number }}" --exclude-contexts 'Renovate|[Cc]odecov|Dependabot|edx-platform-ci' --all-checks
- name: Merge pull request
env:
GIT_TOKEN: ${{ secrets.requirements_bot_github_token }}
run: |
merge_pull_request --org "${{ env.org_name }}" --repo "${{ env.repo_name }}" --pr_number
merge_pr.py --org "${{ env.org_name }}" --repo "${{ env.repo_name }}" --pr_number "${{ env.pr_number }}"

0 comments on commit 34c0000

Please sign in to comment.