Skip to content

feat: Automatically dismiss stale reviews on pull request open/synchronize#41

Closed
HarunamiYaki wants to merge 1 commit intoTypeslint:developmentfrom
HarunamiYaki:development
Closed

feat: Automatically dismiss stale reviews on pull request open/synchronize#41
HarunamiYaki wants to merge 1 commit intoTypeslint:developmentfrom
HarunamiYaki:development

Conversation

@HarunamiYaki
Copy link
Copy Markdown
Member

Description

This pull request introduces a new feature to automatically dismiss stale reviews when a pull request is opened or synchronized. In this context, "stale reviews" are considered those that have been submitted with "Requested Changes" or "Approved" status. In addition to introducing the new dismissal feature, this pull request refactors the code for a more concise. The refactored logic now utilizes the prsLabels constant to streamline label checking.

@typeslint-cli
Copy link
Copy Markdown
Contributor

typeslint-cli Bot commented Nov 10, 2023

Hello @HarunamiYaki Thank you for submitting Pull Request, please wait for next notification after we review your Pull Request

@typeslint-cli typeslint-cli Bot added Core Pending Unread Issues/PRs labels Nov 10, 2023
@Muunatic Muunatic added this to the v3.0.0 milestone Nov 10, 2023
@Muunatic Muunatic marked this pull request as draft November 10, 2023 16:44
@Muunatic
Copy link
Copy Markdown
Member

i had to convert it to draft status to testing out this build, if everything looks good i'll approve this pr quickly, thanks!

@Muunatic Muunatic added the Minor label Nov 10, 2023
@Muunatic Muunatic marked this pull request as ready for review November 11, 2023 19:22
@Muunatic
Copy link
Copy Markdown
Member

everything looks good to go. i'm planning to schedule the approval on monday to make sure everything is clear, thanks! @HarunamiYaki

Copy link
Copy Markdown
Member

@Muunatic Muunatic left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
Contributor

@typeslint-cli typeslint-cli Bot left a comment

Choose a reason for hiding this comment

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

@HarunamiYaki your pull request has been approved by [MAINTAINER]@Muunatic, please type Ready to merge for merging

@typeslint-cli typeslint-cli Bot added Approved Approved PRs and removed Pending Unread Issues/PRs labels Nov 13, 2023
@Muunatic
Copy link
Copy Markdown
Member

Superseded #42

@Muunatic Muunatic closed this Nov 13, 2023
@Typeslint Typeslint locked as resolved and limited conversation to collaborators Nov 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants