Skip to content

Add a skeleton of the 3.5.0 release post. #95

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

Merged
merged 16 commits into from
Jul 1, 2016

Conversation

fables-tales
Copy link
Member

No description provided.

@@ -0,0 +1,125 @@
---
Copy link
Member

Choose a reason for hiding this comment

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

The file name should have 3-5, not 3.5 -- at least if we want to follow the pattern for all the other release posts.

Copy link
Member

Choose a reason for hiding this comment

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

In fact, I had to rename the file to get middleman to serve this blog post locally. I guess it requires the dash form.

That suggests that you wrote this without seeing how it renders in middleman. That's fine, but be sure to check how it renders in your browser. I've seen plenty of cases where things did not render how you'd expect initially, and the manual check is important before we publish.

Copy link
Member Author

Choose a reason for hiding this comment

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

done

@myronmarston
Copy link
Member

It's a great start, @samphippen. Nice work!

Let me know when you've rename the file and then I can get started filling in some of the notable changes.

@yujinakayama -- are you planning to do a Japanese translation again?


RSpec 3.5 has just been released! Given our commitment to
[semantic versioning](http://semver.org/), this should be a trivial
upgrade for anyone already using any RSpec 3 release, but if we did introduce
Copy link
Member

Choose a reason for hiding this comment

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

s/any/an/ any reads funny to me

@fables-tales
Copy link
Member Author

I renamed the file and GitHub collapsed all the feedback. I'll address the rest of it tomorrow. @myronmarston do you want to add any points that you have in the mean time?

@myronmarston
Copy link
Member

I pushed my first round of updates. Will do more later.

@myronmarston
Copy link
Member

OK, I pushed some more updates. I've written up everything I had planned. Anyone want to review?

As far as I know, this is ready to publish besides the stuff we'll fill in or replace on the day of release...except for one thing. The file name indicates a date of 2016-06-14, which isn't accurate. @samphippen, feel free to update that now or wait until release day.

@@ -0,0 +1,345 @@
---
title: RSpec 3.5 has been released!
author: Sam Phippen
Copy link
Member

Choose a reason for hiding this comment

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

Since I wrote up a bunch of the notable changes section, should we change this to Sam Phippen and Myron Marston?

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

Copy link
Member

Choose a reason for hiding this comment

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

Done.

@JonRowe
Copy link
Member

JonRowe commented Jun 16, 2016

I've added a snippet about the new respond_to matcher stuff

has lacked support for placing expectations upon keyword arguments in a variety of our matchers.

In rspec-expectations 3.5 we add support for checking whether an object responds to a method
using keyword arguments. You'll now be able to check
Copy link
Member

Choose a reason for hiding this comment

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

You'll now be able to check

Seems like this sentence is truncated?

has lacked support for placing expectations upon keyword arguments in a variety of our matchers.

In rspec-expectations 3.5 we add support for checking whether an object responds to a method
using keyword arguments. You'll now be able to check a methods response signature for particular
Copy link
Member

Choose a reason for hiding this comment

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

s/methods/method's/


~~~ ruby
expect(my_object).to respond_to(:find).with_keywords(:limit, :offset) }
expect(my_object).to respond_to(:find).with(1).argument.and_keywords(:limit, :offset)
Copy link
Member

Choose a reason for hiding this comment

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

One thing that's unclear to me in this example (and that perhaps warrants changes in rspec-expectations before we are locked into the semantics) is what it means to mix an argument count and keywords. Is the (1).argument bit specifying the number of non-keyword arguments (e.g. positional arguments)? Where it gets confusing is that at the call site keyword args are just passed as a single hash arg, which is then interpreted as keyword args or as a single positional arg depending on how the method is defined. So it's not clear to me if keyword args should be counted and if so, if they should be counted as one arg (since the underlying mechanism uses a single hash) or as one arg per keyword.

Copy link
Member

Choose a reason for hiding this comment

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

Argument in this sense means a traditional unnamed argument.

@myronmarston
Copy link
Member

Any update on the release date for rails 5, @samphippen?

@fables-tales
Copy link
Member Author

The release got knocked back, so we don't need to do our release yet.

@myronmarston
Copy link
Member

The release got knocked back, so we don't need to do our release yet.

I figured that, since I saw no news about Rails 5 being released. Is there a new ETA?

@fables-tales
Copy link
Member Author

I will find out and get back to you.

@fables-tales fables-tales merged commit 70446d7 into source Jul 1, 2016
@fables-tales fables-tales deleted the samphippen/3-5-0-release-post branch July 1, 2016 16:13
@yujinakayama
Copy link
Member

@yujinakayama -- are you planning to do a Japanese translation again?

Doing now :)

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