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

Update jekyll and bundle versions in gemspec #25

Merged
merged 2 commits into from
Jan 6, 2020
Merged

Update jekyll and bundle versions in gemspec #25

merged 2 commits into from
Jan 6, 2020

Conversation

jordigarcl
Copy link
Contributor

Trying to use this theme as a gem on a fresh jekyll install results in the following error:

Bundler could not find compatible versions for gem "jekyll":
  In Gemfile:
    jekyll (~> 4.0.0)

    garth-jekyll-theme (~> 1.0.4) was resolved to 1.0.4, which depends on
      jekyll (~> 3.6)

This PR fixes the issue by updating the jekyll depencency.

Trying to use this theme as a gem on a fresh jekyll install results in the following error:

```
Bundler could not find compatible versions for gem "jekyll":
  In Gemfile:
    jekyll (~> 4.0.0)

    garth-jekyll-theme (~> 1.0.4) was resolved to 1.0.4, which depends on
      jekyll (~> 3.6)
```

This PR fixes the issue by updating the jekyll depencency.
@daviddarnes daviddarnes self-requested a review January 5, 2020 17:43
@daviddarnes
Copy link
Owner

Hey @jordigarcl! Thanks for using my theme and for providing a fix. Nice spot with the jekyll version issue, I don't suppose you can try out setting the jekyll version to the following:

  spec.add_runtime_dependency "jekyll", ">= 3.6", "< 5.0"

This should widen the scope of compatibility rather than bumping it up

@jordigarcl
Copy link
Contributor Author

jordigarcl commented Jan 6, 2020

@daviddarnes I've tried your suggestion and it seems to work fine. I've done the same for the bundler dependency, which was also giving me issues with my local 2.1.2version.

PD: This is a very nice theme!

@daviddarnes daviddarnes merged commit d3c4b6a into daviddarnes:master Jan 6, 2020
@daviddarnes
Copy link
Owner

Awesome stuff, thanks for the fix and for the kind words!

@jordigarcl jordigarcl deleted the fix-version branch January 6, 2020 20:18
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.

None yet

2 participants