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.

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

Trigger add-task-to-project when PR marked ready for review #3273

Merged
merged 2 commits into from
Sep 24, 2024

Conversation

diegoreymendez
Copy link
Contributor

@diegoreymendez diegoreymendez commented Sep 13, 2024

Task/Issue URL: https://app.asana.com/0/1208036161861763/1207931637636063/f

Description

Trigger add-task-to-project when PR marked ready for review

Testing

You can reproduce these steps by creating a copy of this branch and opening a PR to see the CI behavior.

  1. This is what was shown in the CI checks when I opened this as a draft without links.
Screenshot 2024-09-13 at 3 42 20 PM
  1. This is what I saw after adding a link, when I changed the status to "Ready for Review"
Screenshot 2024-09-13 at 3 45 21 PM

Definition of Done:


Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

@diegoreymendez diegoreymendez self-assigned this Sep 13, 2024
@diegoreymendez diegoreymendez requested a review from dus7 September 13, 2024 13:44
@diegoreymendez diegoreymendez marked this pull request as ready for review September 13, 2024 13:45
@dus7 dus7 mentioned this pull request Sep 16, 2024
Copy link
Contributor

@dus7 dus7 left a comment

Choose a reason for hiding this comment

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

Add Task to the App Board Project step is cancelled on the ready_for_review transition. I think the line causing that is

if: ${{ github.event.action == 'opened' && steps.check-board-membership.outputs.task_in_project == '0' }}

Here it's verified the workflow was initiated by PR open event, to not spam the board when task link was edited. I think adding an event condition in this line should be enough, like so:

(github.event.action == 'opened' || github.event.action == 'ready_for_review') && (...)

Tested in #3279.

@dus7 dus7 marked this pull request as draft September 20, 2024 09:27
@dus7 dus7 marked this pull request as ready for review September 20, 2024 09:28
dus7 added a commit to duckduckgo/iOS that referenced this pull request Sep 24, 2024
Task/Issue URL:
https://app.asana.com/0/414235014887631/1207931637636063/f
Tech Design URL:
CC:

**Description**:

Trigger add-task-to-project when PR marked ready for review.

Counterpart of duckduckgo/macos-browser#3273.

**Steps to test this PR**:

* Create a copy of this branch and open a draft PR. 
* Mark as ready for review and check if the "Add Task to App Board
Project" action is successful.

**Definition of Done (Internal Only)**:

* [ ] Does this PR satisfy our [Definition of
Done](https://app.asana.com/0/1202500774821704/1207634633537039/f)?

---
###### Internal references:
[Software Engineering
Expectations](https://app.asana.com/0/59792373528535/199064865822552)
[Technical Design
Template](https://app.asana.com/0/59792373528535/184709971311943)
@dus7 dus7 merged commit 423f347 into main Sep 24, 2024
22 checks passed
@dus7 dus7 deleted the diego/ci-add-task-to-project branch September 24, 2024 09:53
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.

2 participants