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

Extract its into external rspec-its gem #1083

Closed
3 tasks done
myronmarston opened this issue Sep 23, 2013 · 12 comments
Closed
3 tasks done

Extract its into external rspec-its gem #1083

myronmarston opened this issue Sep 23, 2013 · 12 comments

Comments

@myronmarston
Copy link
Member

  • Build rspec-its gem
  • Remove its from rspec-core's master branch
  • Add deprecation notice to rspec-core's 2.99 branch
@palfvin
Copy link
Contributor

palfvin commented Sep 23, 2013

Per our email exchange, I'm up for taking this on.

@myronmarston
Copy link
Member Author

Sounds good. I just created a repo for you:

https://github.com/rspec/rspec-its

You should have commit access.

As far as next steps, see the checklist listed on this issue above. It might be useful to peruse these to see how we've done it for rspec-collection_matchers:

Let us know if you have any concerns or questions.

Thanks!

@palfvin
Copy link
Contributor

palfvin commented Sep 23, 2013

Thanks for getting me set up, Myron.
On Sep 23, 2013 1:37 PM, "Myron Marston" notifications@github.com wrote:

Sounds good. I just created a repo for you:

https://github.com/rspec/rspec-its

You should have commit access.

As far as next steps, see the checklist listed on this issue above. It
might be useful to peruse these to see how we've done it for
rspec-collection_matchers:

Let us know if you have any concerns or questions.

Thanks!


Reply to this email directly or view it on GitHubhttps://github.com//issues/1083#issuecomment-24952256
.

@palfvin
Copy link
Contributor

palfvin commented Sep 27, 2013

Myron, I haven't transferred anything to rspec/rspec-its or generated any pull requests, but I have the rspec-core changes and rspec-its set up and tests in my own repository. Do you want to review them there first? A lot of the tools stuff is new for me, including any non-trivial use of github, so I wanted to make my initial mistakes as locally as possible.

The actual extraction seemed trivial, though, so unless I missed something, hopefully there's not a lot to worry about in that regard. Let me know.

@palfvin
Copy link
Contributor

palfvin commented Sep 27, 2013

Just reread your 3.0 release plan and realized that I didn't address the 2.99 needs. Will do that.

@myronmarston
Copy link
Member Author

Myron, I haven't transferred anything to rspec/rspec-its or generated any pull requests, but I have the rspec-core changes and rspec-its set up and tests in my own repository. Do you want to review them there first? A lot of the tools stuff is new for me, including any non-trivial use of github, so I wanted to make my initial mistakes as locally as possible.

Just submit PRs to the main repos, please. Do them one at a time: we're not going to consider removing its from rspec-core until the gem is ready, and we don't need the deprecation warning in 2.99 until its removed from master.

@palfvin
Copy link
Contributor

palfvin commented Sep 28, 2013

Just submitted rspec/rspec-its#1 for step 1 of creating the gem.

JonRowe added a commit that referenced this issue Oct 3, 2013
Remove "its" support as part of rspec-core issue #1083
@palfvin
Copy link
Contributor

palfvin commented Oct 3, 2013

@JonRowe I'm trying to test against rspec 2.99 as you indicated, but I'm running into trouble trying to get a 2.99 reference to work. If I specify each of the core gems individually in my Gemfile, using the 2-99-maintenance branch, then it works fine. But I specify that same branch for the rspec meta gem, I got an error that it can't resolve the dependency on the 2.99.0.pre components. It's not a big deal, obviously, but I was just curious if I was missing something.

@JonRowe
Copy link
Member

JonRowe commented Oct 4, 2013

Use the 3 gems individually. I don't think there is one for the meta gem.

JonRowe added a commit that referenced this issue Oct 10, 2013
Deprecate 'its' in favor of rspec-its gem as part of #1083
@myronmarston
Copy link
Member Author

Looks like this is done. Closing.

@palfvin
Copy link
Contributor

palfvin commented Oct 17, 2013

@myronmarston Any guidance on how to handle Relish for this extracted gem? Do you want to limit the RSpec publisher to just the gems supported by the core team? If so, do you have any suggestions for what to name the publisher I create for this purpose? Should I just use my own name?

More generally, can you rationalize the use of the rspec github account for this gem when it's not used for, say, xavier's rspec-fire gem? Will this gem have some special status going forward vs. any other gem created to complement the RSpec gems maintained by the core team?

Finally, as I was checking around for other, non-core RSpec gems, I encountered rspec-api-documentation, which seems to have violated the rubygems naming convention involving use of dashes and underscores as covered in http://guides.rubygems.org/name-your-gem/ Is this anything you think is worth trying to raise with the author?

@myronmarston
Copy link
Member Author

Any guidance on how to handle Relish for this extracted gem?

That's completely up to you as maintainer of this gem.

Do you want to limit the RSpec publisher to just the gems supported by the core team?

Yes.

If so, do you have any suggestions for what to name the publisher I create for this purpose? Should I just use my own name?

That's what most people do.

More generally, can you rationalize the use of the rspec github account for this gem when it's not used for, say, xavier's rspec-fire gem? Will this gem have some special status going forward vs. any other gem created to complement the RSpec gems maintained by the core team?

rspec-its has been extracted from gems maintained by the core team. We thought it made sense to make it part of the rspec account on github so that it's clear to others that it is in some sense "blessed" by the core team. It gives it a bit more "legitimacy", if that makes sense.

Finally, as I was checking around for other, non-core RSpec gems, I encountered rspec-api-documentation, which seems to have violated the rubygems naming convention involving use of dashes and underscores as covered in http://guides.rubygems.org/name-your-gem/ Is this anything you think is worth trying to raise with the author?

Nope. It's not my job or anyone else's to police how folks name their gems.

yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this issue Oct 6, 2021
Deprecate 'its' in favor of rspec-its gem as part of rspec/rspec-core#1083

---
This commit was imported from rspec/rspec-core@0928c69.
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

No branches or pull requests

3 participants