-
Notifications
You must be signed in to change notification settings - Fork 62
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
first phase rails upgrade #986
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great and works well on my machine also; smoke tested a variety of behaviors including enketo rendering
note i had to rm -rf tmp
to resolve an occasional bootsnap crash
@@ -86,12 +86,15 @@ jobs: | |||
LINT_RESULT=$? | |||
yarn run test | |||
JEST_RESULT=$? | |||
bundle exec rspec --format RSpec::Github::Formatter --format documentation | |||
bundle exec rspec --tag ~flapping --format RSpec::Github::Formatter --format documentation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
@@ -3,8 +3,6 @@ | |||
# For details on the DSL available within this file, | |||
# see https://guides.rubyonrails.org/routing.html | |||
ELMO::Application.routes.draw do | |||
mount(JasmineRails::Engine => "/specs") if defined?(JasmineRails) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥
@@ -148,8 +163,8 @@ GEM | |||
babel-transpiler (0.7.0) | |||
babel-source (>= 4.0, < 6) | |||
execjs (~> 2.0) | |||
base64 (0.1.1) | |||
bigdecimal (3.1.6) | |||
base64 (0.2.0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops i missed this: the prod error from before has come back, i'm gonna try to revert this specific upgrade
No description provided.