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.

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

Added It.IsAnyValue #40

Merged
merged 2 commits into from
Apr 3, 2013
Merged

Added It.IsAnyValue #40

merged 2 commits into from
Apr 3, 2013

Conversation

Pjanssen
Copy link
Contributor

I've added a method to the It class which creates a match for any value of type TValue. IsAnyValue is different from IsAny in that it does not match null.

@kzu
Copy link
Contributor

kzu commented Mar 31, 2013

How about calling it It.IsNotNull?

@Pjanssen
Copy link
Contributor Author

Works for me!

@Pjanssen
Copy link
Contributor Author

Pjanssen commented Apr 2, 2013

Should I change it in my fork and submit a new pull request? Or can you handle it on your side?

@kzu
Copy link
Contributor

kzu commented Apr 2, 2013

change it on your fork. If the pull request was sent from the fork, and not
a specific commit, you don't need to do anything else

/kzu

Daniel Cazzulino

On Tue, Apr 2, 2013 at 8:01 AM, Pier Janssen notifications@github.comwrote:

Should I change it in my fork and submit a new pull request? Or can you
handle it on your side?


Reply to this email directly or view it on GitHubhttps://github.com//pull/40#issuecomment-15768779
.

@Pjanssen
Copy link
Contributor Author

Pjanssen commented Apr 2, 2013

Done! Also moved the documentation to make it align with the rest of the code/docs.

kzu added a commit that referenced this pull request Apr 3, 2013
Added It.IsNotNull<T>.
@kzu kzu merged commit 44dd387 into devlooped:dev Apr 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants