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

README uses expect() syntax #782

Merged
merged 1 commit into from Jan 31, 2013

Conversation

banyan
Copy link
Contributor

@banyan banyan commented Jan 31, 2013

In the future, we plan to change the defaults so that only expect is available unless you explicitly enable should.

http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax

If this plan is progressing, README should use expect syntax for rspec newbie. WDYT?
Thanks.

@alindeman
Copy link
Contributor

Yes! Thank you for doing this.

alindeman added a commit that referenced this pull request Jan 31, 2013
@alindeman alindeman merged commit 89fce7c into rspec:master Jan 31, 2013
@banyan
Copy link
Contributor Author

banyan commented Jan 31, 2013

@alindeman

Thanks for quick response!!
And I have one more question.

Today I was asked by my colleague, and he said
"Which is better to use, should syntax or expect syntax? This documentation (https://www.relishapp.com/rspec/docs/gettingstarted) seems to me to use should syntax."

So my question is, is there any way to fix (or pull request) relish's documentation?

@myronmarston
Copy link
Member

We just haven't gotten around to updating it. Want to take a stab at it?

@banyan
Copy link
Contributor Author

banyan commented Feb 1, 2013

Yes! I'll do it if I can!

@stephenmckinney
Copy link

👍 I've fielded similar questions from RSpec newbies

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

4 participants