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

Does Not Work Out Of The Box #17

Open
firezdog opened this issue May 31, 2022 · 2 comments
Open

Does Not Work Out Of The Box #17

firezdog opened this issue May 31, 2022 · 2 comments
Assignees

Comments

@firezdog
Copy link

firezdog commented May 31, 2022

I wouldn't normally consider it an issue for an application not to work out of the box when pulling down the github repo, but the readme doesn't have any instructions / suggestions concerning installation of dependencies. Just doing the most naive pull of the repo and putting the suggested command into my cli, I get the following error:

Traceback (most recent call last):
        4: from ./agent_q/bin/agent_q:3:in `<main>'
        3: from ./agent_q/bin/agent_q:3:in `require_relative'
        2: from ./agent_q/lib/agent_q.rb:4:in `<top (required)>'
        1: from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require': cannot load such file -- capybara (LoadError)

If there is some required version of ruby etc. that needs to be installed to run the application, either an install script should be created or instructions on what is required should be included in the readme.

@firezdog
Copy link
Author

It seems like this will work with an installation of Chrome in say a Docker container -- but a small modification to Capybara was required to allow the browser to run with no sandbox (at least for Docker). The README mentions using / installing PhantomJS but does not elaborate on what needs to be done with it -- in the Docker setup I mentioned we didn't seem to need PhantomJS at all (unless it is already being installed by the Gem).

@epugh
Copy link
Member

epugh commented Jun 13, 2022

thanks for this! I'd love a PR with a better README ;-)

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

No branches or pull requests

2 participants