Skip to content
This repository has been archived by the owner on Nov 17, 2022. It is now read-only.

Test for and document undocumented potential ValueErrors #241

Merged
merged 1 commit into from
Nov 15, 2017

Conversation

hayfield
Copy link
Contributor

Some, but not all, functions in iati.default had it documented and tested that the error could be raised.
This makes the documentation and testing more comprehensive.

Some, but not all, functions in iati.default had it documented and tested that the error could be raised.
This makes the documentation and testing more comprehensive.
@hayfield hayfield added complete A PR that is in a state that is ready for review. docs Changes to documentation. tests Relating to tests and testing. labels Nov 15, 2017
@hayfield hayfield requested a review from a team November 15, 2017 12:14
Copy link
Contributor

@allthatilk allthatilk left a comment

Choose a reason for hiding this comment

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

🚀

@hayfield hayfield merged commit c6dca3d into dev Nov 15, 2017
@hayfield hayfield deleted the default-invalid-version branch November 15, 2017 12:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
complete A PR that is in a state that is ready for review. docs Changes to documentation. tests Relating to tests and testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants