Add list of abandoned organizations #95
Conversation
larsxschneider
force-pushed
the
lars/old-orgs
branch
2 times, most recently
from
6e44db6 to
7c2d22d
Compare
| ></canvas> | ||
| <div class="info-box"> | ||
| <p> | ||
| An organizations is considered <em>abandoned</em> if none of its |
There was a problem hiding this comment.
I’d suggest not wrapping these lines manually and leaving this to the editor program. My argument is that diffs tend to become very ugly when flow text is manually rewrapped later on.
There was a problem hiding this comment.
true. but diffs for very long lines are ugly too. I dunno
| repositories). | ||
| </p> | ||
| <p> | ||
| If the content has still value, then you could <a href="https://help.github.com/articles/archiving-repositories/">archive</a> |
There was a problem hiding this comment.
I’d write “still has value” instead of “has still value.”
| query = ''' | ||
| SELECT | ||
| users.login AS "organization", | ||
| date(MAX(pushes.created_at)) AS "last push" |
There was a problem hiding this comment.
Shouldn’t DATE be in uppercase for consistency?
| JOIN users ON repositories.owner_id = users.id | ||
| JOIN pushes ON pushes.repository_id = repositories.id | ||
| WHERE | ||
| users.type = "organization" |
There was a problem hiding this comment.
The indentation looks off by 3 spaces to me.
| GROUP BY | ||
| users.id | ||
| HAVING | ||
| MAX(pushes.created_at) < DATE_SUB(NOW(), INTERVAL 1 YEAR) |
There was a problem hiding this comment.
I’d prefer using the Python-based approach to selecting time ranges instead of using DATE_SUB(NOW()) and INTERVAL 1 YEAR.
The main reason for this is that I think it would be better to exclude today’s data from the report and to start with yesterday instead.
Find the organizations that have not received a push for the longest time. Only look at organizations that have not received a push for at least one year. Only look at repositories that are still maintained (not archived!).
larsxschneider
force-pushed
the
lars/old-orgs
branch
from
7c2d22d to
a63950b
Compare
|
@pluehne please review, again |
pluehne
changed the title
Find the organizations that have not received a push for the longest time.
Only look at organizations that have not received a push for at least one
year.