Commit 086af44
committed
fix: use repo.owner.login for security-updates URLs and handle datetime in created_after filter
## What/Why
Fix two bugs exposed by the PyGithub migration: (1) repo.owner is now a
NamedUser object, not a string, so security-updates URLs rendered as
"repos/NamedUser(login=...)/..." and 404'd silently; (2) repo.created_at
is a datetime object but is_repo_created_date_before called
fromisoformat() on it, which raises TypeError.
## Proof it works
All 180 tests pass including 2 new tests for datetime input to
is_repo_created_date_before (both before and after filter date).
## Risk + AI role
Medium -- the repo.owner.login fix touches the security-updates code
path which is pragma: no cover. AI-generated (Claude Opus 4.6) with
human review.
## Review focus
Whether repo.owner.login is the correct attribute for all PyGithub
Repository objects (org-owned vs user-owned repos).
Signed-off-by: jmeridth <jmeridth@gmail.com>1 parent 6e1b200 commit 086af44
2 files changed
Lines changed: 30 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
213 | 213 | | |
214 | 214 | | |
215 | 215 | | |
216 | | - | |
| 216 | + | |
217 | 217 | | |
218 | 218 | | |
219 | | - | |
| 219 | + | |
220 | 220 | | |
221 | 221 | | |
222 | 222 | | |
| |||
300 | 300 | | |
301 | 301 | | |
302 | 302 | | |
303 | | - | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
304 | 306 | | |
305 | | - | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
306 | 313 | | |
307 | 314 | | |
308 | 315 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
| 6 | + | |
6 | 7 | | |
7 | 8 | | |
8 | 9 | | |
| |||
801 | 802 | | |
802 | 803 | | |
803 | 804 | | |
| 805 | + | |
| 806 | + | |
| 807 | + | |
| 808 | + | |
| 809 | + | |
| 810 | + | |
| 811 | + | |
| 812 | + | |
| 813 | + | |
| 814 | + | |
| 815 | + | |
| 816 | + | |
| 817 | + | |
| 818 | + | |
| 819 | + | |
| 820 | + | |
| 821 | + | |
| 822 | + | |
804 | 823 | | |
805 | 824 | | |
806 | 825 | | |
| |||
0 commit comments