Skip to content

netiquette - Suggest to open discussion if possible#9076

Merged
fregante merged 4 commits into
mainfrom
suggest-disc
Apr 4, 2026
Merged

netiquette - Suggest to open discussion if possible#9076
fregante merged 4 commits into
mainfrom
suggest-disc

Conversation

@SunsetTechuila
Copy link
Copy Markdown
Member

@SunsetTechuila SunsetTechuila commented Mar 11, 2026

+ addresses #9028 (comment)

Test URLs

eslint/eslint#20304

#6 (nothing changed)

Screenshot

image image

Comment thread source/github-helpers/index.ts
@SunsetTechuila SunsetTechuila changed the title netiquette - suggest to open discussion if possible netiquette - Suggest to open discussion if possible Mar 11, 2026
Copy link
Copy Markdown
Member

@fregante fregante left a comment

Choose a reason for hiding this comment

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

Eh, not a fan of discussions in general, but I guess some repo may prefer them.

To be honest I'd wait until someone complains before adding this. What if you add "open discussion" only when issues are disabled?

Comment thread source/features/netiquette.tsx
Comment thread source/github-helpers/index.ts
'.GlobalNav',
// Remove after June 2026
'.js-repo-nav',
].join(',')}`;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Parens missing. This is why I dislike template literals for short strings 🥹

I'd extract this to its own selector helper function, like is('div', 'span') === ':is(div,span)'

@SunsetTechuila
Copy link
Copy Markdown
Member Author

What if you add "open discussion" only when issues are disabled?

Do you know any such repos? I only know about the ones where issues are automatically closed if they were created without prior discussion, but that's not something we can check.

I think this PR should be merged as is or not merged at all.

Copy link
Copy Markdown
Member

@fregante fregante left a comment

Choose a reason for hiding this comment

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

Let's also exclude the feature on GHE, in a separate PR

@fregante fregante merged commit 07f8cfd into main Apr 4, 2026
9 checks passed
@fregante fregante deleted the suggest-disc branch April 4, 2026 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants