Skip to content

Update to react 16 #792

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

Merged
merged 24 commits into from
Oct 9, 2017
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
dccc8de
Correct all deprecation warnings in addons
BookOfGreg Sep 27, 2017
373f45a
Update React and React-dom to 16
BookOfGreg Sep 27, 2017
68c9ed2
Prebundle React16 and new addons
BookOfGreg Sep 27, 2017
c616ee5
Update generator to React 16 proptypes and ES6 export
BookOfGreg Sep 29, 2017
c87d890
Include react-create-class in generator for non-es6
BookOfGreg Sep 29, 2017
3cf34fb
Update test fixtures to use createReactClass
BookOfGreg Sep 29, 2017
cb66e31
Migrate more tests to React16, add createReactClass to dummyw3
BookOfGreg Sep 30, 2017
7e450ab
Add reactCreateBuild to addons
BookOfGreg Sep 30, 2017
32c9df3
Replace therubyracer with mini_racer
BookOfGreg Sep 30, 2017
e594694
Add PropTypes to default react bundle
BookOfGreg Sep 30, 2017
896b4fc
Update README on PropType
BookOfGreg Sep 30, 2017
6b0b878
Webpacker error changed, dont yell at JS errors
BookOfGreg Sep 30, 2017
57f7adc
Stop scanning for React in JSX
BookOfGreg Sep 30, 2017
61462c5
Merge update-to-chromedriver-for-js-tests into update-to-react-16
BookOfGreg Sep 30, 2017
aa65c48
Replace React.DOM with React.createElement
BookOfGreg Sep 30, 2017
4b73b9f
Add Proptypes to React with addons
BookOfGreg Oct 1, 2017
8ab2c7b
Remove Addons distributables
BookOfGreg Oct 1, 2017
1ba1110
Stop checking for reactid as createElement doesnt make one
BookOfGreg Oct 1, 2017
18faeed
Remove final references to React.createClass
BookOfGreg Oct 1, 2017
a8a9cd2
Give Webpacker ES6 import always
BookOfGreg Oct 1, 2017
8883be9
Dont reference react where it can be compiled
BookOfGreg Oct 1, 2017
691d2aa
Touch a file after a second to make it compile on fast computers
BookOfGreg Oct 1, 2017
0c1e60c
Merge resolved
BookOfGreg Oct 2, 2017
d7adca6
Reorder appraisal file so I get the most interesting failures first
BookOfGreg Oct 2, 2017
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
119 changes: 60 additions & 59 deletions Appraisals
Original file line number Diff line number Diff line change
@@ -1,86 +1,41 @@
appraise 'rails-3.2' do
gem 'rails', '~> 3.2.21'
gem 'rack-cache', '~> 1.6.1'
# Modern turbolinks depends on `Rails.application.assets` which no-worky
gem 'turbolinks', '~> 2.0'
end

appraise 'rails-4.0.5' do
# Depends on sprockets-rails ~> 2.0.0. Support for
# `Rails.application.assets_manifest` which is used by
# `ServerRendering::ManifestContainer` has only been added in
# sprockets-rails 2.2.2. Ensure that server rendering falls back to
# `ServerRendering::EnvironmentContainer`.`
gem 'rails', '4.0.5'
gem 'turbolinks'
end

appraise 'rails-4.0-with-mini_racer' do
gem 'rails', '~> 4.0.13'
gem 'mini_racer', :platforms => :mri
gem 'therubyrhino', :platforms => :jruby
gem 'turbolinks'
end

appraise 'rails-4.1' do
gem 'rails', '~> 4.1.10'
# Just to make sure we support old Turbolinks:
gem 'turbolinks', '~> 2.3.0'
end

appraise 'rails-4.2-sprockets_2' do
gem 'rails', '~> 4.2.1'
gem 'sprockets', '~> 2.12'
gem 'turbolinks'
end

appraise 'rails-4.2-sprockets_3' do
gem 'rails', '~> 4.2.1'
gem 'sprockets', '~> 3.5'
gem 'turbolinks', '~> 2.5.0'
end

appraise 'rails-4.2-sprockets_4' do
gem 'rails', '~> 4.2.1'
appraise 'rails-5.1-sprockets_4' do
gem 'rails', '~> 5.1'
gem 'sprockets', '~> 4.0.x'
gem 'turbolinks', '~> 2.5.0'
# This ExecJS backend provides stateful context
# which the default nodejs backend does not
gem 'mini_racer', :platforms => :mri
gem 'turbolinks', '~> 5.0.0'
end

# no_sprockets is a magical name from sprockets_helper.rb in test to
# load in certain tests or not.
appraise 'rails-5_no_sprockets_webpacker_1_1' do
appraise 'rails-5_no_sprockets_webpacker_3' do
gem 'rails', '~> 5.0.0'
gem 'webpacker', '~> 1.1.0'
gem 'webpacker', '>= 3.0'
# This ExecJS backend provides stateful context
# which the default nodejs backend does not
gem 'mini_racer', :platforms => :mri
gem 'therubyrhino', :platforms => :jruby
end

appraise 'rails-5_no_sprockets_webpacker_1_x' do
appraise 'rails-5_no_sprockets_webpacker_2' do
gem 'rails', '~> 5.0.0'
gem 'webpacker', '~> 1.2'
gem 'webpacker', '~> 2.0'
# This ExecJS backend provides stateful context
# which the default nodejs backend does not
gem 'mini_racer', :platforms => :mri
gem 'therubyrhino', :platforms => :jruby
end

appraise 'rails-5_no_sprockets_webpacker_2' do
appraise 'rails-5_no_sprockets_webpacker_1_x' do
gem 'rails', '~> 5.0.0'
gem 'webpacker', '~> 2.0'
gem 'webpacker', '~> 1.2'
# This ExecJS backend provides stateful context
# which the default nodejs backend does not
gem 'mini_racer', :platforms => :mri
gem 'therubyrhino', :platforms => :jruby
end

appraise 'rails-5_no_sprockets_webpacker_3' do
appraise 'rails-5_no_sprockets_webpacker_1_1' do
gem 'rails', '~> 5.0.0'
gem 'webpacker', '>= 3.0'
gem 'webpacker', '~> 1.1.0'
# This ExecJS backend provides stateful context
# which the default nodejs backend does not
gem 'mini_racer', :platforms => :mri
@@ -95,8 +50,54 @@ appraise 'rails-5-no_sprockets' do
gem 'rails', '~> 5.0.0'
end

appraise 'rails-5.1-sprockets_4' do
gem 'rails', '~> 5.1'
appraise 'rails-4.2-sprockets_4' do
gem 'rails', '~> 4.2.1'
gem 'sprockets', '~> 4.0.x'
gem 'turbolinks', '~> 5.0.0'
gem 'turbolinks', '~> 2.5.0'
# This ExecJS backend provides stateful context
# which the default nodejs backend does not
gem 'mini_racer', :platforms => :mri
end

appraise 'rails-4.2-sprockets_3' do
gem 'rails', '~> 4.2.1'
gem 'sprockets', '~> 3.5'
gem 'turbolinks', '~> 2.5.0'
end

appraise 'rails-4.2-sprockets_2' do
gem 'rails', '~> 4.2.1'
gem 'sprockets', '~> 2.12'
gem 'turbolinks'
end

appraise 'rails-4.1' do
gem 'rails', '~> 4.1.10'
# Just to make sure we support old Turbolinks:
gem 'turbolinks', '~> 2.3.0'
end

appraise 'rails-4.0-with-mini_racer' do
gem 'rails', '~> 4.0.13'
gem 'mini_racer', :platforms => :mri
gem 'therubyrhino', :platforms => :jruby
gem 'turbolinks'
end

appraise 'rails-4.0.5' do
# Depends on sprockets-rails ~> 2.0.0. Support for
# `Rails.application.assets_manifest` which is used by
# `ServerRendering::ManifestContainer` has only been added in
# sprockets-rails 2.2.2. Ensure that server rendering falls back to
# `ServerRendering::EnvironmentContainer`.`
gem 'rails', '4.0.5'
gem 'turbolinks'
end

appraise 'rails-3.2' do
gem 'rails', '~> 3.2.21'
gem 'rack-cache', '~> 1.6.1'
# Modern turbolinks depends on `Rails.application.assets` which no-worky
gem 'turbolinks', '~> 2.0'
end

5 changes: 0 additions & 5 deletions Rakefile
Original file line number Diff line number Diff line change
@@ -32,13 +32,8 @@ namespace :react do
task :copy do
environments = ['development', 'production']
environments.each do |environment|
# Without addons:
copy_react_asset("#{environment}/react-browser.js", "#{environment}/react.js")
copy_react_asset("#{environment}/react-server.js", "#{environment}/react-server.js")

# With addons:
copy_react_asset("#{environment}/react-browser-with-addons.js", "#{environment}-with-addons/react.js")
copy_react_asset("#{environment}/react-server-with-addons.js", "#{environment}-with-addons/react-server.js")
end
end

Loading