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 code contributors from git history #160

Closed
wants to merge 33 commits into from

Conversation

yevhenorlov
Copy link
Member

@yevhenorlov yevhenorlov commented Feb 26, 2020

Issue: #161

@sudodoki
Copy link
Member

sudodoki commented Feb 26, 2020

btw, is it just automatically generating list of committers into the repo? I'm wondering because there was definitely some code from https://github.com/MashaTimmpa/3_kottans at one point (she was the one responsible for some previous redesign, IIRC)

@yevhenorlov
Copy link
Member Author

btw, is it just automatically generating list of committers into the repo? I'm wondering because there was definitely some code from https://github.com/MashaTimmpa/3_kottans at one point (she was the one responsible for some previous redesign, IIRC)

At this step I'm adding only contributors from git history. (see #161 for roadmap).

Currrently all-contributors-cli does not provide that functionality, so I'm trying to get data from github using suggestions in this comment.

Of course, if there's some contributions that are not visible in that data, we should add them manually.

]
},
{
"login": "m1n0s",
Copy link
Member

Choose a reason for hiding this comment

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

I see a lot of people, who did not contributed into new website.
What the background of this PR?

Copy link
Member Author

Choose a reason for hiding this comment

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

We've discussed at some point that for a lot of people who contributed to Kottans their work is not visible, because it was removed from current state of the repo.

While that was a necessary decision, I think we still should recognize their past work. IMO this is a good place to do that.

Kottans are a community with a long history, that should be visible 💪

Copy link
Member

Choose a reason for hiding this comment

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

As far as I see this right now, we have core, lectors and mentors, who mentioned on a website.

Contributers into this repo are contributers into website, who spent a lot of personal time and effort into it. I don't see any reason why we should bring everyone here. Probably we should find a better way to do this.

Pls, make an RFC in kottans_ongoing_projects, we should discuss, how to mention everyone.

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure, will do. Closing this for now

Copy link
Member

Choose a reason for hiding this comment

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

@zonzujiro most of those contributions were created when mentors were adding themselves to mentors data yaml

Copy link
Member

Choose a reason for hiding this comment

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

@sudodoki this is exactly what we are discussing right now #161 (comment) :)

@yevhenorlov yevhenorlov changed the title WIP: Add all contributors Add code contributors from git history Feb 26, 2020
@zonzujiro zonzujiro deleted the add-all-contributors branch February 27, 2020 10:38
Sign up for free to 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