asok / projectile-rails

Emacs Rails mode based on projectile
258 stars 59 forks source link

Deleted tag master #111

Closed Silex closed 7 years ago

Silex commented 7 years ago

Hello,

I took the liberty of deleting the tag "master". This was very likely a mistake.

screenshot from 2017-03-14 09-56-12

Silex commented 7 years ago

Also, I wanted to fix the tests but I'm not sure what to do. Should the test be modified or the command be fixed?

  Scenario: Running rails console
    Given I open the app file "app/models/user.rb"
    And I turn on projectile-rails-mode
    When I run "projectile-rails-console"
    Then I should be in buffer "*ruby*"
    And projectile-rails should be turned on
    When I type "1 + 1"
    And I open the app file "app/models/user.rb"
    When I run "projectile-rails-console"
    Then I should be in buffer "*ruby*"
      Expected to be in buffer '*ruby*', but was in '*ruby*<2>'
    And I should see "1 + 1"
asok commented 7 years ago

Hi, thanks for taking the action on the incorrect tag. The error is due to the fact that inf-ruby now supports multiple irb buffers. Instead of looking for the precise buffer name *ruby* I would look for a regexp *ruby*(<[0-9]+>)? (actually it's not a valid elisp regexp, I just wrote it to let you know what I mean).

Silex commented 7 years ago

Alright, so I'd fix the test. I'll make a PR.

Silex commented 7 years ago

You pushed the tag master again. Please run this in your local repo:

git tag -d master
git fetch --all
git tag

The last line should show that no tag named master exists.

The test were failing again because it tried to build the tag master. I pushed a somewhat dummy commit effectively repairing the tests because it didn't try to build the tag master anymore :-)

asok commented 7 years ago

Thanks for having an eye on me on that :) I've deleted that on 1 machine. I'm out of home but once I'm back I'll delete it also on the machine I have in my home.

Silex commented 7 years ago

Can we close this?

asok commented 7 years ago

Most certainly yes.