Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update approve-trigger.yml #1300

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
+2 −0
Open

Update approve-trigger.yml #1300

Changes from 8 commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
d6c1af1
Update approve-trigger.yml
aderemi1224 Oct 7, 2024
a3823c8
Update approve-trigger.yml
aderemi1224 Oct 7, 2024
2047033
Newly Updated approve-trigger.yml
aderemi1224 Oct 7, 2024
1ebe594
Update approve-trigger.yml
aderemi1224 Oct 7, 2024
13b9de6
Update approve-trigger.yml
aderemi1224 Oct 7, 2024
aed976e
Update approve-trigger.yml
aderemi1224 Oct 7, 2024
a3a5257
Update approve-trigger.yml
aderemi1224 Oct 9, 2024
81a38b5
Update approve-trigger.yml
aderemi1224 Oct 9, 2024
06394b6
Merge branch 'master' into master
DhanshreeA Oct 10, 2024
ab9b308
Update approve-trigger.yml
aderemi1224 Oct 10, 2024
9aaa522
Merge branch 'master' into master
DhanshreeA Oct 12, 2024
5c312b2
Update approve-trigger.yml
aderemi1224 Oct 13, 2024
c5d8383
Merge branch 'master' into master
DhanshreeA Oct 14, 2024
a1fe790
Update pr_check.yml
aderemi1224 Oct 14, 2024
8c2e65c
Update pr_check.yml
aderemi1224 Oct 14, 2024
9051c35
Update pr_check.yml
aderemi1224 Oct 14, 2024
6aace2c
Update pr_check.yml
aderemi1224 Oct 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 11 additions & 9 deletions .github/workflows/approve-trigger.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,16 +2,18 @@ name: Approve Command Trigger
on:
issue_comment:
types: [created]

jobs:
approve-command-trigger:
DhanshreeA marked this conversation as resolved.
Show resolved Hide resolved
if: contains(github.event.issue.labels.*.name, 'Model Request')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The label is actually called new-model, and not Model Request.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Refer here, for example: https://github.com/ersilia-os/ersilia/issues?q=is%3Aopen+is%3Aissue+label%3Anew-model

runs-on: ubuntu-latest
steps:
- name: Approve Command Trigger
uses: peter-evans/slash-command-dispatch@13bc09769d122a64f75aa5037256f6f2d78be8c4 # pin@v4.0.0
with:
token: ${{ secrets.PAT }}
issue-type: issue
permission: write
reactions: false
commands: |
approve
- name: Approve Command Trigger
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should keep the single tab indentation

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aderemi1224 can you please restore the indentation?

uses: peter-evans/slash-command-dispatch@13bc09769d122a64f75aa5037256f6f2d78be8c4 # pin@v4.0.0
with:
token: ${{ secrets.PAT }}
issue-type: issue
permission: write
reactions: false
commands: |
approve