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 the 3.5.0.beta1 release post. #92

Merged
merged 1 commit into from
Feb 6, 2016
Merged

Conversation

fables-tales
Copy link
Member

No description provided.

@fables-tales
Copy link
Member Author

@rspec/rspec can someone review and deploy this asap?

@fables-tales
Copy link
Member Author

@JonRowe can I also get a release bit on this app?

@myronmarston
Copy link
Member

I'm making breakfast but plan to review this after that.

On Sat, Feb 6, 2016 at 6:53 AM, Sam Phippen [email protected]
wrote:

@JonRowe https://github.com/JonRowe can I also get a release bit on
this app?


Reply to this email directly or view it on GitHub
#92 (comment).

tests in Rails 5. RSpec has been affected by being downstream of this, but
fortunately, we were able to make the process relatively smooth for our users.

If you have existing specs with `:type => :view` or `:type => :controller`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's slightly confusing that Rails' deprecation of controller tests affect view specs. Not sure if it's worth spending any effort explaining why but I found this surprising.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's because controller specs and view specs use some of the same code paths. Nobody who tests without RSpec uses view tests, so rails isn't really talking about them. The announced deprecations of controller specs also cover view specs.

@myronmarston
Copy link
Member

It's a good blog post, @samphippen. Thanks for leading the 3.5 release!

@fables-tales fables-tales force-pushed the 3-5-0-beta1-release-post branch from 20d8b4c to 090ea06 Compare February 6, 2016 19:02
@fables-tales
Copy link
Member Author

@myronmarston is this good to merge?

@myronmarston
Copy link
Member

LGTM

fables-tales pushed a commit that referenced this pull request Feb 6, 2016
Add the 3.5.0.beta1 release post.
@fables-tales fables-tales merged commit 97553c0 into source Feb 6, 2016
@fables-tales fables-tales deleted the 3-5-0-beta1-release-post branch February 6, 2016 19:24
@fables-tales
Copy link
Member Author

@rspec/rspec how do we roll this out?

@myronmarston
Copy link
Member

The Readme explains how to deploy.

@myronmarston
Copy link
Member

Your gemfile snippet isn't rendering right, @samphippen:

screen shot 2016-02-06 at 5 51 53 pm

Can you fix it? You need to use a slightly different syntax for code blocks than github flavored markdown.

@yujinakayama
Copy link
Member

Fixed in 39d3ef9.

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.

3 participants