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

Add Vue.js, Facebook, ... #8

Closed
maxdevjs opened this issue Nov 9, 2018 · 4 comments
Closed

Add Vue.js, Facebook, ... #8

maxdevjs opened this issue Nov 9, 2018 · 4 comments

Comments

@maxdevjs
Copy link
Contributor

maxdevjs commented Nov 9, 2018

Added a few orgs to my fork:

Hints, suggestions, is this done the right way, useful...? :)

@bnb
Copy link
Member

bnb commented Nov 9, 2018

Hey, thanks for the work!

Some feedback:

  • In your searches, you don't need to include sort:updated-desc since that's included in the query to @octokit/rest.js.
  • Facebook is definitely an interesting idea, and one I'd not thought about! I really like the concept, though I think we may want to have a slightly more comprehensive description because it's an org with multiple facets 😄

@maxdevjs
Copy link
Contributor Author

maxdevjs commented Nov 9, 2018

Thanks for clarifications.

  • ops... about sort:updated-desc I did a paste & copy of electron project, not node, should electron (and gutenberg) be changed too?
  • I agree, again... had this doubt too. And what would be better? Add orgs as a whole (example: org:facebook and not just repo:facebook/react, as per org:nodejs) or specific repos (repo:angular/angular, as per repo:microsoft/vscode, etc)?

May be could take multiple command line arguments?

good-first-issue.js angular search in org
good-first-issue.js angular angular search in repo (!? hard to distinguish)
good-first-issue.js react (harder to implement?) Vs good-first-issue.js facebook react search in repo (?!)

Edit. looks like #10 is a further step...

@bnb
Copy link
Member

bnb commented Nov 13, 2018

Indeed, #10 has now landed.

I'd like to keep things centralized into specific commands. I'm not opposed to having unique commands for facebook, react, jest, and so on.

Node.js is an org but it's a weird org in that everything in it is around Node.js as a project... so they're all basically the same project. Someone who is going to be interested in a first contribution to Node.js can contribute to any of the repos.

Facebook has multiple projects that span different technologies... someone who wants to contribute to React probably isn't interested in contributing to facebook/mysql-5.6.

Separating those concerns out and having per-project commands makes sense IMO... regardless of whether or not a project is an org or a repo. Other examples of orgs that are projects I can think of:

  • Babel
  • Electron (hey, there's already a command for this one!)
  • webpack
  • Gatsby

That said, I think it's a nifty a idea and something we should explore to have org commands like facebook, microsoft, github, google, airbnb, and so on!

Perhaps this syntax could be used for diverse (rather than single-focus) orgs:

# Syntax
good-first-issue <org>/<repo>

# Examples
good-first-issue facebook/react
good-first-issue microsoft/azuredatastudio
good-first-issue github/gitignore

Does that make sense?

@bnb
Copy link
Member

bnb commented Jul 25, 2019

Going to close this 👍

@bnb bnb closed this as completed Jul 25, 2019
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

No branches or pull requests

2 participants