Skip to content

Conversation

@sebastienros
Copy link
Member

@sebastienros sebastienros commented Nov 24, 2025

Prevents policies from re-assigning it when value is changed later

Microsoft Reviewers: Open in CodeFlow

Prevents policies from re-assigning it when value is changed later
@sebastienros sebastienros requested a review from a team as a code owner November 24, 2025 23:45
Copilot AI review requested due to automatic review settings November 24, 2025 23:45
@joperezr joperezr merged commit e4b118f into main Nov 24, 2025
8 checks passed
@joperezr joperezr deleted the sebastienros-patch-1 branch November 24, 2025 23:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR attempts to restrict the automatic assignment of pull request authors to only occur when PRs are first created (opened), rather than on every PR update. However, the implementation contains a critical bug.

Key Issues

  • The isAction: action: Opened condition was added to the wrong policy rule, breaking the intended functionality

Comment on lines +161 to +162
- isAction:
action: Opened
Copy link

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

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

The isAction: action: Opened condition is added to the wrong rule. This is part of the Issue_Comment rule (lines 159-170) that removes the 'waiting-author-feedback' label, not the Pull_Request rule (lines 154-158) that assigns PRs to authors.

To fix this issue and achieve the PR's goal of "assign authors only on PR creation", move these two lines to be part of the previous rule. The corrected structure should be:

- if:
  - payloadType: Pull_Request
  - isAction:
      action: Opened
  then:
  - assignTo:
      author: True
  description: Assign PRs to authors

Copilot uses AI. Check for mistakes.
This was referenced Dec 11, 2025
This was referenced Dec 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants