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.

Sign up for GitHub

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

Jump to bottom

Rails 6 upgrades (do not merge) #84

Open

nwalberts wants to merge 3 commits into master from rails-6-upgrades

Contributor

nwalberts commented Nov 4, 2021

Existing work done in order to upgrade make_it_so generator to Rails 6

Unfortunately, the architecture of a Rails 6 app is different enough that I am putting this on pause

Things to investigate prior to continuing

investigate using webpacker as new asset pipeline manager over sprockets, and what changes will be needed accordingly
figure out if we want sprockets-rails 4 or to just leave it off and have all assets managed by webpacker
work with consulting team on babel/webpack configuration to update the config that we add with make_it_so (we might be able to remove much of our old patterns now that webpacker comes standard with Rails 6)

And while we are at it

remove shoulda configuration
remove enzyme/karma configuration

nwalberts added 3 commits

November 4, 2021 15:47


          config changes to allow generation of rails 6 app

7b1eca2


          change default thor ruby version to 3.0.2

c3768eb


          update asset management for new sprockets verion

936cd42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment