Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't automatically recommend creating an issue on DefinitelyTyped #331

Merged
merged 1 commit into from Sep 28, 2022

Conversation

orta
Copy link
Collaborator

@orta orta commented Jun 30, 2021

👋 Because the request will basically just be ignored - 1st we use Discussions for there's sorts of requests, and 2nd no-one is probably going to do the work for you because that work doesn't scale to DT size. We get one or two of these a day, might as well just not recommend them.

}

const { repo, org } = vcsDetails
if (repo === "DefinitelyTyped" && org === "DefinitelyTyped") {
Copy link

Choose a reason for hiding this comment

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

I wouldn't like such PR to be merge, blocking any repository. Where do we stop people suggesting blocking their repos?

The problem is solved by the community, and encouraging better collaboration, and discipline. This doesn't solve the problem properly either scales as a fix.

Copy link

Choose a reason for hiding this comment

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

Also, something super clear, I strongly agree with the underline sentiment and intent, but I strongly disagree with the proposed fix.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I mean DT is pretty much the biggest public collection of npm packages, I'm sure it's easy to say "well DT has ~7,000 packages to cover"

Copy link

Choose a reason for hiding this comment

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

Sorry Orta, I am not sure what you are trying to coney right now, would you mind expanding on that?

@orta
Copy link
Collaborator Author

orta commented Sep 28, 2022

I chatted with David about this, and it's good to go - yordis, you're welcome to read up about what definitely typed is: https://github.com/DefinitelyTyped/DefinitelyTyped but the issues this feature raises on that repo are spam and waste maintainers time

@orta orta merged commit 639b9c8 into ds300:master Sep 28, 2022
@ds300
Copy link
Owner

ds300 commented Oct 25, 2022

This was just released in v6.5.0! Thanks for your contribution 🙏🏼 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants