Skip to content

Conversation

@romanzow
Copy link

@romanzow romanzow commented Feb 21, 2019

Resolves #331


This change is Reviewable

@justin808
Copy link
Member

@thetonyrom This looks great. However, we really should take out the QT dependency and remove https://github.com/shakacode/react-webpack-rails-tutorial/blob/master/Gemfile#L97...

There are a few other references to capybara-webkit in the repo.

We're actually going beyond headless chrome and using cypress.io for testing.

@romanzow
Copy link
Author

@justin808 I see your point. I have a pretty busy week, probably, I'll update the PR in the coming weekend.

@romanzow
Copy link
Author

romanzow commented Mar 3, 2019

@justin808 I've updated the PR, have a look

@justin808
Copy link
Member

I'm about to make a new PR that fixes your CI issue. rebase on top of that and push again, please.

@justin808
Copy link
Member

@thetonyrom Can you rebase on top of master and push.

@romanzow
Copy link
Author

Done

@romanzow
Copy link
Author

Hey @justin808 . Let's finish it! :) Let me know if anything.

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

Labels

None yet

2 participants