Skip to content

Conversation

patrickhulce
Copy link
Collaborator

Summary
All ya gotta do to have the bot close these issues in the future is add "pending-close" and "variability"

Current set of issues: https://github.com/GoogleChrome/lighthouse/issues?q=is%3Aopen+is%3Aissue+label%3Avariability+label%3Apending-close

Related Issues/PRs
#10657

@patrickhulce patrickhulce requested a review from a team as a code owner April 28, 2020 20:58
@patrickhulce patrickhulce requested review from brendankenny and removed request for a team April 28, 2020 20:58
Copy link
Contributor

@brendankenny brendankenny left a comment

Choose a reason for hiding this comment

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

LGTM!

body: 'Thanks! Appreciate you filing this bug. :clap:


This is a known issue, most well described in #10657. So, **we''ll automatically
Copy link
Contributor

Choose a reason for hiding this comment

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

weird way to escape ' but ok yaml :P

labels:
$and:
- $includes: pending-close
- $includes: variability
Copy link
Contributor

Choose a reason for hiding this comment

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

I feel like this is going to auto-trigger at some point for an issue that's about variability but shouldn't be deduped and then someone's going to add pending-close when the issue is wrapping up and it's going to be all confusing, but I guess that's unlikely to happen often :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

heh, maybe we'll see :)

@patrickhulce patrickhulce merged commit e5b6ed2 into master Apr 29, 2020
@patrickhulce patrickhulce deleted the variability_bot branch April 29, 2020 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants