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

Fix building issues #118

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

ShamoX
Copy link

@ShamoX ShamoX commented Aug 26, 2020

Some of the build fail is from the fact that new rails app seems to have defaults some options that are not compatible with --skip-bundle.
rails/rails#28916
So I had to add few other --skip-* options to make it work like the tests expects it to work

Roland Laurès added 4 commits August 26, 2020 18:49
Squashed commit of the following:

commit 50b5f12
Author: Roland Laurès <[email protected]>
Date:   Wed Aug 26 18:45:30 2020 +0200

    Add new needed parameter because of --skip-bundle
Squashed commit of the following:

commit fecc5c8
Author: Roland Laurès <[email protected]>
Date:   Wed Aug 26 23:36:10 2020 +0200

    Fix Chromiumdriver start and connexion

commit 6fd87e8
Author: Roland Laurès <[email protected]>
Date:   Wed Aug 26 23:21:41 2020 +0200

    Use webdrivers gem instead of chromedriver-helper

commit 2acad93
Author: Roland Laurès <[email protected]>
Date:   Wed Aug 26 18:44:01 2020 +0200

    Just add a gem to help debug

commit 4e83d27
Author: Roland Laurès <[email protected]>
Date:   Wed Aug 26 18:13:53 2020 +0200

    Adding debug port to Chrome driver
@ShamoX ShamoX changed the title Fix 'rails new testapp' in feature/installation Fix building issues Aug 26, 2020
@ShamoX
Copy link
Author

ShamoX commented Aug 26, 2020

Needed to upgrade minor ruby versions for 2.5 which was failing missing URI::Generic

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.

1 participant