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

all-contributors generate #152

Closed
githubsaturn opened this issue Jan 21, 2019 · 7 comments
Closed

all-contributors generate #152

githubsaturn opened this issue Jan 21, 2019 · 7 comments

Comments

@githubsaturn
Copy link

  • all-contributors-cli version: 5.10.2
  • node version: v10.15.0
  • npm (or yarn) version: 6.4.1

Isn't all-contributors generate supposed to generate the list of contributors automatically?

#!/bin/sh

git clone [email protected]:caprover/caprover.git
cd caprover
npm install --save-dev all-contributors-cli
npx all-contributors init
npx all-contributors check
npx all-contributors generate
cat README.md

What you did:

Ran the script above and I was expecting that contributors are automatically added to README.md after running those commands. However, the npx all-contributors generate just exits and nothing gets added to README.md

npx all-contributors check works fine. It prints all missing contributors on screen, however, npx all-contributors generate does not do anything.

This issue is similar to #93.
But it's not the same, because the additional ...ALL-CONTRIBUTORS-LIST:START... stuff are added to README.md

@jakebolam jakebolam reopened this Jan 21, 2019
@jakebolam
Copy link
Collaborator

@githubsaturn (sorry for the accidental close, on my phone)

Generate will generate the list of contributors based on the .all-contributorsrc file. (PRs welcome to make the docs clearer on this)

init is intended for new projects, but does not automatically fetch the list of contributors (yet).

If your interested in contributing to an auto-generated/fetch feature there is some discussion in #117.

@jakebolam
Copy link
Collaborator

Just curious (to help us strive to continue to build better tooling). Did you by chance come across the new bot? Any reason for choosing the CLI over the bot?

@githubsaturn
Copy link
Author

Just curious (to help us strive to continue to build better tooling). Did you by chance come across the new bot? Any reason for choosing the CLI over the bot?

Exactly because of issues like this one. I generally don't trust a bot to come and make changes for me unless it's a very well known bot. As a rule of thumb, I'd like to inspect what is being done closely.

@githubsaturn
Copy link
Author

PS: I close this issue. As it's not a bug. It's working as intended.

I was under the impression that the premise of this project is to autogenerate the contributor list for you - similar to Open Collective. See this one for example: https://github.com/parcel-bundler/parcel#contributors

@jakebolam
Copy link
Collaborator

Exactly because of issues like this one. I generally don't trust a bot to come and make changes for me unless it's a very well known bot. As a rule of thumb, I'd like to inspect what is being done closely.

Fair enough. The entire source code for the bot (as well as infrastructure etc) is all completely open source.

@jakebolam
Copy link
Collaborator

I was under the impression that the premise of this project is to autogenerate the contributor list for you - similar to Open Collective. See this one for example: https://github.com/parcel-bundler/parcel#contributors

Oh yeah, that makes sense. We've held off for a while on this feature, as all-contributors is intended to recognise all contributors ✨ not just those who code. We understand there is a barrier of entry on existing projects, and will hopefully have something in the future to help automate this.

@Berkmann18
Copy link
Member

@jakebolam For the auto-generation, there's some brainstorming in AC-CLI#117 and one other issue where I posted this classification diagram which is what I was planning on using to try something out (whenever I'll be done with the intensive course I'm currently doing).

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

3 participants