Skip to content
This repository has been archived by the owner on Aug 9, 2024. It is now read-only.

Fixes https://github.com/wiki-ai/revscoring/issues/121 #143

Merged
merged 4 commits into from
Jul 29, 2015

Conversation

halfak
Copy link
Member

@halfak halfak commented Jul 28, 2015

…ta_if_exists' to handle the situations when there is no parent revision or previous user revision.

Fixes #121

…ta_if_exists' to handle the situations when there is no parent revision or previous user revision.
@halfak halfak changed the title Adds new processor function to API extractor 'process_revision_metada… Fixes https://github.com/wiki-ai/revscoring/pull/143 Jul 28, 2015
@adamwight
Copy link
Contributor

Looks like an improvement, but I donno if the stuff consuming Datasources can handle Nones.

@@ -17,3 +17,10 @@ def test_seconds_since():
previous_user_revision.metadata: FakeRevisionMetadata(Timestamp(1))
}
eq_(solve(seconds_since, cache=cache), 9)

# Makes sure we don't crash when there was not previous user revision
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"there was no"?

@he7d3r he7d3r changed the title Fixes https://github.com/wiki-ai/revscoring/pull/143 Fixes https://github.com/wiki-ai/revscoring/issues/121 Jul 29, 2015
he7d3r added a commit that referenced this pull request Jul 29, 2015
@he7d3r he7d3r merged commit 2cf8a57 into master Jul 29, 2015
@halfak halfak deleted the new_page_error branch August 8, 2015 17:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants