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

Add missing redcarpet dependency #136

Closed
wants to merge 1 commit into from

Conversation

jamietanna
Copy link

While investigating #135, I noticed that the build requires the
redcarpet gem, otherwise we get the error:

[error]: Missing 'redcarpet' gem for Markdown formatting. Install it
with `gem install redcarpet`

While investigating auth0#135, I noticed that the build requires the
redcarpet gem, otherwise we get the error:

    [error]: Missing 'redcarpet' gem for Markdown formatting. Install it
    with `gem install redcarpet`
@joshcanhelp
Copy link
Contributor

@jamietanna - Thanks so much for digging in on this!

Are you seeing that error when you run bundle exec rake documentation? I'm not seeing that error locally or in CI. I'm also not seeing this used anywhere in this gem.

@jamietanna
Copy link
Author

Interesting - I'm seeing that when running:

ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-linux]
Bundler version 1.16.6

@joshcanhelp
Copy link
Contributor

Is this within the gem or in your own project? I'm on the same version of Rails but downgraded Bundler to 1.16.2 to match the CI. It's curious because, like I said, I don't see anything in the gem that uses redcarpet or Markdown in general.

@jamietanna
Copy link
Author

Sorry, this is within just this project. I'll verify against different versions of Ruby in Docker, to see if there's anything I've broken on the host.

@joshcanhelp
Copy link
Contributor

No activity ... happy to re-open if you discover that this makes a difference.

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