Skip to content
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

Chore/update appraisals #934

Merged
merged 15 commits into from
Mar 31, 2019
Merged

Chore/update appraisals #934

merged 15 commits into from
Mar 31, 2019

Conversation

BookOfGreg
Copy link
Member

Summary

Adds Rails 5.2 + Webpacker 3, 4.pre versions to appraisal, with Webpacker 4.pre set as allow fail.
Closes #918

Appraisals Outdated Show resolved Hide resolved
@BookOfGreg
Copy link
Member Author

@ttanimichi Appears to be some failing test unrelated to this appraisal change. Must say in general I'm finding Travis a little flakey at times but this one does seem like a real failure on Webpacker 3.

@BookOfGreg
Copy link
Member Author

BookOfGreg commented Mar 30, 2019

This grid in Travis is roughly what I'm aiming for:

Gemfiles\Rubies 2.6.2 2.5.3 2.3.8 jruby-9.1.9.0
rails_4.2_sprockets_4 x
rails_5_no_sprockets_webpacker_1_1 x
rails_5_no_sprockets_webpacker_1_x x
rails_5_no_sprockets_webpacker_2 x
rails_5_no_sprockets_webpacker_3 x
rails_5.2_no_sprockets_webpacker_3 x
rails_5.2_no_sprockets_webpacker_4 x x x Allow Fail
rails_4.2_sprockets_2 x
rails_4.2_sprockets_3 x
rails_5_no_sprockets x
rails_5.1_sprockets_4 x
rails_5.2_sprockets_4 x x x Allow Fail

@BookOfGreg
Copy link
Member Author

Reverted as Ruby 2.6.2 can't seem to gem install webdrivers

Parsing documentation for childprocess-0.9.0
Before reporting this, could you check that the file you're documenting
has proper syntax:

  /Users/me/.rbenv/versions/2.6.2/bin/ruby -c lib/childprocess/tools/generator.rb

RDoc is not a full Ruby parser and will fail when fed invalid ruby programs.

The internal error was:

	(NoMethodError) undefined method `[]' for nil:NilClass

ERROR:  While executing gem ... (NoMethodError)
    undefined method `[]' for nil:NilClass

@BookOfGreg BookOfGreg merged commit 4a46a4c into master Mar 31, 2019
@BookOfGreg BookOfGreg deleted the chore/update-appraisals branch March 31, 2019 18:49
@0xdevalias
Copy link
Contributor

0xdevalias commented Apr 29, 2019

With this merged now, does that mean webpacker 4.x is actually supported? If so.. should the README get updated to say so?

Ref: #983

@BookOfGreg
Copy link
Member Author

@0xdevalias thanks for asking for clarity on this.

Webpacker 4.x is supported in that it didn't crash the tests. I haven't personally tried it yet as my time is limited.
If you vouch it currently works (or can report bugs specifically for Webpacker 4.x) then yes we can call it supported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add more CIs (Rails 5.2, Webpacker v4 pre)
3 participants