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

Add Boolean AND, OR operators (&, |) #537

Merged
merged 1 commit into from Apr 22, 2014

Conversation

@yelled3
Copy link

@yelled3 yelled3 commented Apr 22, 2014

We can consider keeping the & and | operators
@myronmarston

see: #505 (comment)

@JonRowe
Copy link
Member

@JonRowe JonRowe commented Apr 22, 2014

I'm ok with this addition, WDYT @myronmarston (given your "consider" above ;))?

@myronmarston
Copy link
Member

@myronmarston myronmarston commented Apr 22, 2014

This LGTM but I want to see the build go green before we merge.

@yelled3
Copy link
Author

@yelled3 yelled3 commented Apr 22, 2014

@myronmarston everything should be green + updated changelog

JonRowe added a commit that referenced this pull request Apr 22, 2014
Add Boolean AND, OR operators (&, |)
@JonRowe JonRowe merged commit 0e5f978 into rspec:master Apr 22, 2014
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
JonRowe added a commit that referenced this pull request Apr 22, 2014
@myronmarston
Copy link
Member

@myronmarston myronmarston commented Apr 22, 2014

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants