Be slightly more defensive when processing annotations for the atom feed. #2345

Merged
merged 1 commit into from Jul 3, 2015

Projects

None yet

3 participants

@nickstenning
Member

When processing annotations for the Atom feed, we need to be slightly
more defensive -- the assumptions we'd like to make about annotation
structure aren't always true.

Fixes https://app.getsentry.com/hypothesis/prod/group/68815541/.

@landscape-bot

Code Health
Code quality remained the same when pulling 203e45b on hypothesis:defensive-feed-construction into c6be6c1 on hypothesis:master.

@nickstenning nickstenning Be slightly more defensive when processing annotations
When processing annotations for the Atom feed, we need to be slightly
more defensive -- the assumptions we'd like to make about annotation
structure aren't always true.

Fixes https://app.getsentry.com/hypothesis/prod/group/68815541/.
7050db2
@landscape-bot

Code Health
Repository health decreased by 0.29% when pulling 7050db2 on defensive-feed-construction into b3d7b3e on master.

@tilgovi
Contributor
tilgovi commented Jul 3, 2015

Thanks. I saw this yesterday, too, but didn't get a chance to finish the patch. 🍻

@tilgovi tilgovi merged commit 3ccc68e into master Jul 3, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@tilgovi tilgovi deleted the defensive-feed-construction branch Jul 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment