Skip to content
This repository has been archived by the owner on Oct 2, 2018. It is now read-only.

Standardize exceptions and params #8

Open
bcanseco opened this issue Jul 4, 2017 · 0 comments
Open

Standardize exceptions and params #8

bcanseco opened this issue Jul 4, 2017 · 0 comments
Labels
refactoring

Comments

@bcanseco
Copy link
Owner

bcanseco commented Jul 4, 2017

If something can't be null, throw an ArgumentNullException.

Also, go through each service and see which actually return an HTTP code on failure (essentially resulting in a thrown HttpRequestException) and which do not. Maybe the custom exceptions can be removed in favor of something built in?
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
refactoring
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bcanseco