Latest less-rails

[veelenga]

Thanks a lot for maintaining this gem.

We use it in one of our production apps which is running on rails 4.2. Upgrading to the latest ruby/rails offers us to upgrade less-rails which is a dependency on less-rails-bootstrap.

This PR is about to upgrade this gem to the latest available less-rails:

metaskills/less-rails@v2.8.0...v3.0.0

This addresses #131 and #130

I hope the PR is ok and you will be able to review/merge it ❤️

[denqxotl]

👍

[ghost]

👍

[Genkilabs]

@veelenga I took a look at the diff and it seems good to me.
Since it has still been several months with no motion on this I'm going to suggest your build at
gem 'less-rails-bootstrap', git: 'https://github.com/veelenga/less-rails-bootstrap', :ref => '7c479c2fdff500dc036c15364aa085332a73c642'
for our use on our staging, pending eventual production deployment (Ruby 2.4.2, Rails 5.1.5). If we encounter any issues I'll post them here.

Thank you for all your work on this issue. Hopefully it is merged into master soon!

UPDATE For those considering using this, the above ref has been running solidly on one of our our production servers at roughly 150req/min since April 8 2018 with no issues.

[NikoRoberts]

Anything the community can do to help get this merged @metaskills ?

[a-m-zill]

Hi @metaskills!

Can this PR merged, please?!

It will be very helpfully for many projects.

Thx

[metaskills]

@simi Any thoughts?

[simi]

If I understand well, we're dropping Rails 3.x and moving to less-rails 3.x resulting into new major release will be needed here. Am I right?

[NikoRoberts]

Yes basically.
We’re now using the PRed fork in production with Rails 5.1.

[derekgottlieb]

Any chance to go ahead and get this merged and a new version pushed?

[aergonaut]

Hi @simi @metaskills. Pilling on here to wonder if there's any possibility for this PR to merged and a new version to be published. My team would really love to see a new version that permits less-rails 3!

[metaskills]

I won't have time to help with this but if I need to grant access to someone then I certainly will. @simi are you working this gem anymore?

[simi]

Sure, I'm still around. I'll take a look.

[ahwagner]

Seeing how this PR started 2 years ago my comment probably won't move the needle, but... any chance this can get merged in sometime soon @metaskills @simi?

@veelenga thanks for this fork.

[Genkilabs]

@metaskills Any chance on a tapping that juicy green merge button?

[metaskills]

Can @simi do that or someone a little more close to maintaining this project?

[simi]

ℹ️ I'm around, but since this PR does more than just updating less-rails dependency, I need to spend some time on that, split the changes, and test (also with latest rails).

🤔 If anyone around is able to open pull request updating just less-rails for now, it will be merged and released much faster.