Closed mfazekas closed 1 year ago
@jejacks0n any feedback on this?
I've tried to fix travis as well, fixed clode climate codecov and used webdrivers gem so it installs geckodriver, etc correctly.
@jejacks0n ping, rails 5.1 is out now
@jejacks0n any update on this?
Alright, I tried to get this merged, but it's complicated and stale. I'm going to take what you've done here and commit it.. sorry!
The code climate adjustments are where I had the most trouble. I know you were trying to be kind with that effort. =)
@jejacks0n no worries i'll try to rebase it.
@jejacks0n i've removed code climate changes. Only the ruby 2.5/rails 6 passes now.
Rails 5 test fails with You requested coverage reports, but no results were found. Are all files being ignored in your coverage config? If you have expand_assets set to false, you will need to remove spec_helper from the ignore list.
Do you have tips for that?
rebased into e1c9097
anyone would wish to see this change being merged? please vote in
Rails 5.1 will introduce webpacker see https://github.com/rails/webpacker.
The ideas is that it allows assets to be compiled by webpacker, but those have to be include by adding
javascript_pack_tag