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

Populate the 'authors' table with unique commit authors. #386

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

veenstra
Copy link
Contributor

This migration script finds the unique commit authors from the "master" branch in all of the known repos and inserts them into the "authors" table if they do not already exist (which they shouldn't, but adding a check was easy and allows the script to be run safely more than once). It also finds matching entries in the "users" table (by email) and fills in the "user_id" field.

@veenstra
Copy link
Contributor Author

Please don't pull this yet. These commits populate the db tables. But I need to add the code that maintains these tables when new commits are ingested. Otherwise, we will have to re-populate the tables.

I will create a new commit and add it to this branch (which will add it to this pull request).

@veenstra
Copy link
Contributor Author

Okay, I believe this is ready to merge now.

@veenstra
Copy link
Contributor Author

Wait, don't pull yet. I want to make a change to the migration that populates the author_id field in the commits table.

@cespare
Copy link
Contributor

cespare commented Jan 17, 2013

Stop playing with my ❤️

@veenstra
Copy link
Contributor Author

Okay, now it's ready.

pretty sure ... :-)

@veenstra
Copy link
Contributor Author

Actually, one more change is coming...

@veenstra
Copy link
Contributor Author

Okay, now it's ready. For sure. I'd bet a nickel on it.

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.

2 participants