Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove validPerson from ExceptionExercises #73

Closed
sanjivsahayamrea opened this issue May 3, 2018 · 2 comments
Closed

Remove validPerson from ExceptionExercises #73

sanjivsahayamrea opened this issue May 3, 2018 · 2 comments
Assignees
@ssanj
Labels
possibly-remove remove to make content easier

Comments

@sanjivsahayamrea
Copy link
Contributor

Should we remove validPerson from ExceptionExercises?

 def createValidPeople: List[Person] = ???

and just use:

def createValidPeople2: List[Person] = ???

We are trying to demonstrate that you have to use hacks when you use Exceptions as values.
@sanjivsahayamrea sanjivsahayamrea added the possibly-remove remove to make content easier label May 3, 2018
@wjlow
Copy link
Owner

wjlow commented May 3, 2018

Yup.

Since createValidPeople is unused. Just skip straight to createValidPeople2 and show them how hard it is.

@ssanj
Copy link
Contributor

ssanj commented May 3, 2018

removed in #82

@wjlow wjlow closed this as completed May 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ssanj ssanj

Labels
possibly-remove remove to make content easier
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ssanj @wjlow @sanjivsahayamrea