GitHub now add a Co-Authored-By to the commit message when you squash the PR on merge. It would be good if we start to understand that to give the authorship to both the committer and the person on Co-Authored-By.

Hi,

I noticed that the alias I added in #127 does not seem to be reflected on the live site. Is this because we haven't deployed the application after that was merged? If so, can we redeploy? If not, did I make a mistake in #127?

Thanks!

I believe the build failures are related to rails/rails#41750.

I'm thinking #178 will fix the build failures, I have not gotten dc/sync to complete successfully on my local machine, but the tests are passing. Someone might want to double-check me on this! 😅

Edit: Got things running and this seems to be working as expected.

I am contributing to Rails since 2019 and it is mapped to alkesh26 (16 commits).

But my recent two commits are mapped to alkeshghorpade.

I have not changed my GitHub handle https://github.com/alkesh26 and all my 18 Rails commits are visible here including the recent ones.

What I remember is I deleted the forked Rails repo under alkesh26 and forked it again. Not sure if this can be the cause.

One example of this is @bogdanvlviv. He is not in the list anymore. I could not find a reason for that but seems like this commit is not in the list, or at least it could not get his name.

rails/rails@f2987ce

The FAQ page contains these two contradictory sentences:

Multiple authors
Git has no support for multiple authors, and again the project has figured out conventions to give credit for one commit to several people.

Multiple authors through Co-authored-by
GitHub has added support for multiple authors by using Co-authored-by annotations, see here for more information.

I suggest combining "Multiple authors" and "Multiple authors through Co-authored-by" into one section that explains how the app handles co-author scenarios both before and after GitHub support was added.

My commits are listed as one of two names:

• Author: connorshea <[email protected]>
• Author: Connor Shea <[email protected]>

These are listed as separate users but have the same email address. Unless I'm forgetting an edge case, I don't see any reason for these not to be listed under the same user.

We use a fork of this for GitLab Contributors, so that's where I noticed it.

Thanks for the awesome app :D

Hi

Today I was checking my contributions on the app and I found that my contributions are split like shown below in the captures:

Trying to find a solution I followed the recommendation on the FAQ section and modified this file and then open the following pull request:

#155

Could you guys please confirm if that is right in order for my contributions to appear only as 'David Auza'?

Thank you very much,

Best regards from Colombia 🇨🇴 ,

David Auza

CircleCI runs the test suite when we merge to master, but apparently as of this writing CircleCI does not support checking PRs.

Guess we have to switch to Travis? What do you think @yhirano55?

Abhishek Yadav has been mapped to abhishek, which in turn has added both of my commits in front of Abhishek Yadav.
Check this commit
87fd14b

Two commits below have been contributed by me:
http://contributors.rubyonrails.org/contributors/abhishek-yadav/commits

My commits out of these are: rails/rails@21f0c58
rails/rails@29712f0

Seems we both had abhishek as handler. So all of my commits are mapped to his.

Please look into this.

It would be nice if I could fetch the total number of contributors in a time window.

Currently, I'm parsing using Nokogiri, but I guess that's fine.

Sample code fwiw:

class Contributors
  attr_accessor :body, :start_date, :end_date, :total
  def initialize(start_date, end_date)
    @start_date = start_date.strftime("%Y%m%d")
    @end_date = end_date.strftime("%Y%m%d")
    @body = fetch
    @total = extract_total
  end

  def url
    "https://contributors.rubyonrails.org/contributors/in-time-window/#{@start_date}-#{@end_date}"
  end

  def fetch
    uri = URI.parse(url)
    body = uri.open.read
    return Nokogiri::HTML(body)
  end

  def extract_total
    xpath = "//span[@class=\"listing-total\"][1]"
    text = @body.xpath(xpath.to_s).first.content
    return text.match(/\d+/)
  end
end

After shipping #274, the commits of @jean-francois-labbe are not listed.

We have to understand why and restore them.

@fxn i am contributing in Rails since last year. I checked my name in contributors list but i think it's not proper listed and also reflecting wrong spelling (surname). May be it's releated to one (me @Gaurav2728) only but can be happen with others so i created issue

See different scrren shot below for more details

.
Correct Name is "Gaurav Sharma"

Showing Wrong Name here. ("Gaurav Sharam")
Also commits are not included with my previous rails commit.

As name is different that's why it's showing single commit.