Skip to content

ci: Tag issues if last commenter is user - #7880

Merged
mydea merged 2 commits into
developfrom
fn/ci-commenter-member
Apr 19, 2023
Merged

ci: Tag issues if last commenter is user#7880
mydea merged 2 commits into
developfrom
fn/ci-commenter-member

Conversation

@mydea

Copy link
Copy Markdown
Member

We talked about this a bit recently, so I took a quick stab into implementing this. This adds a label `Status: User responded' when a user comments, and removes it when a member responds. In theory, we should be able to use this label to find issues where we forgot to answer.

@mydea
mydea requested a review from a teamApril 17, 2023 14:27
@mydeamydea self-assigned this Apr 17, 2023
@mydea
mydea requested review from AbhiPrasad and lforst and removed request for a teamApril 17, 2023 14:27
&& github.event.comment.author_association != 'OWNER'
uses: actions-ecosystem/action-add-labels@v1
with:
labels: 'Status: User responded'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we use "Waiting for: Team"?

Then we can eventually rename "Status: Needs more information" to "Waiting for: User".

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I like that

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

changed it! 🚀

@mydea
mydeaforce-pushed the fn/ci-commenter-member branch from 6dd7d9a to 9be0b5dCompareApril 18, 2023 08:39
@mydea
mydea marked this pull request as ready for review April 18, 2023 08:39
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize
@sentry/browser - ES5 CDN Bundle (gzipped + minified)21.04 KB (+0.02% 🔺)
@sentry/browser - ES5 CDN Bundle (minified)65.66 KB (0%)
@sentry/browser - ES6 CDN Bundle (gzipped + minified)19.59 KB (-0.01% 🔽)
@sentry/browser - ES6 CDN Bundle (minified)58.12 KB (0%)
@sentry/browser - Webpack (gzipped + minified)21.19 KB (0%)
@sentry/browser - Webpack (minified)69.07 KB (0%)
@sentry/react - Webpack (gzipped + minified)21.21 KB (0%)
@sentry/nextjs Client - Webpack (gzipped + minified)49.05 KB (0%)
@sentry/browser + @sentry/tracing - ES5 CDN Bundle (gzipped + minified)28.62 KB (+0.01% 🔺)
@sentry/browser + @sentry/tracing - ES6 CDN Bundle (gzipped + minified)26.85 KB (+0.01% 🔺)
@sentry/replay ES6 CDN Bundle (gzipped + minified)45.35 KB (+0.01% 🔺)
@sentry/replay - Webpack (gzipped + minified)39.28 KB (0%)
@sentry/browser + @sentry/tracing + @sentry/replay - ES6 CDN Bundle (gzipped + minified)64.29 KB (+0.01% 🔺)
@sentry/browser + @sentry/replay - ES6 CDN Bundle (gzipped + minified)57.26 KB (+0.01% 🔺)

@mydea
mydea merged commit e6fb47b into developApr 19, 2023
@mydea
mydea deleted the fn/ci-commenter-member branch April 19, 2023 09:32
Sign up for freeto 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

@mydea@chadwhitacre@lforst@Lms24