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

Added rspec-mode #3067

Closed
wants to merge 1 commit into from
Closed

Added rspec-mode #3067

wants to merge 1 commit into from

Conversation

dcluna
Copy link
Contributor

@dcluna dcluna commented Sep 18, 2015

Based on pezra's rspec-mode.

@robbyoconnor
Copy link
Contributor

This is in the ruby layer -- why do we need this? Just put it in the ruby layer...This doesn't need a new layer... so 👎 -- put it in the ruby layer and rebase.

@robbyoconnor
Copy link
Contributor

Also remove the files that do absolutely nothing -- and remove code that does absolutely nothing...

@CestDiego
Copy link
Contributor

he's got a point, just add this to the ruby layer :)

@robbyoconnor
Copy link
Contributor

This still hasn't been addressed...@dcluna :P

@dcluna
Copy link
Contributor Author

dcluna commented Oct 8, 2015

I only added this layer because ruby-test-mode's keybindings weren't working for me, so I'd prefer to keep the alternative test commands. And you're absolutely right, adding a new layer is unnecessary, hope it's better now.

@robbyoconnor
Copy link
Contributor

It's good :)

"mpm" 'rspec-verify-method
"mpt" 'rspec-toggle-example-pendingness
))))

Copy link
Contributor

Choose a reason for hiding this comment

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

Why the mp prefix? Shouldn't this be under the mt prefix per the conventions

@robbyoconnor
Copy link
Contributor

In addition to this _ALWAYS ADD A README WITH THE KEYBINDINGS!_

@robbyoconnor
Copy link
Contributor

@dcluna

I only added this layer because ruby-test-mode's keybindings weren't working for me

Why not try to fix it? Perhaps you didn't have them installed properly...or the ruby you were using didn't have the gem available. I suspect this is the case. In addition, see the line comment.

@robbyoconnor
Copy link
Contributor

Also, rename this as it is currently inaccurately titled ;)

@dcluna dcluna changed the title Added rspec layer Added rspec-mode Oct 8, 2015
@robbyoconnor
Copy link
Contributor

Rebase again ;)

@StreakyCobra
Copy link
Contributor

Now that #3761 has been merged, this one is probably useless and can be closed ☺️

@syl20bnr
Copy link
Owner

syl20bnr commented Dec 6, 2015

Thank you !
I adapted this PR since #3761 has been merged, I added the key bindings of this PR in commit e9684f3
Cherry-picked into develop branch, you can safely delete your branch.

@syl20bnr syl20bnr closed this Dec 6, 2015
@dcluna dcluna deleted the rspec-mode branch December 7, 2015 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants