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

Spider option should autogenerate the spider.yml file if the imported… #504

Merged

Conversation

ThomasWunderlich
Copy link

Bugfix for issue #498, this configures the spider option so that the imports file is autogenerated if it doesn't exist

@ThomasWunderlich
Copy link
Author

Looks like the build is erroring out because the current version of nokogiri requires ruby >= 2.1.0. Do you want to pin nokogiri to an older version or should I upgrade the ruby version?

@ChrisBAshton
Copy link
Contributor

Thanks for bringing that to my attention, @ThomasWunderlich - I have hopefully fixed the build in #505. Could you merge master into your branch so that the tests are run against this PR again?

@ThomasWunderlich
Copy link
Author

@ChrisBAshton ,
Thanks for the quick fix. Wraith is now building with my fix

@ChrisBAshton
Copy link
Contributor

Thanks - your fix looks good, I think we intended to pass the imports_must_resolve boolean as part of v4.0.0 and it never made it in.

Will merge and release as 4.0.1.

@ChrisBAshton ChrisBAshton merged commit 9382fc7 into bbc:master Jan 24, 2017
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.

2 participants