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

Generalize https filter #124

Closed
wants to merge 2 commits into from

Conversation

rymohr
Copy link
Contributor

@rymohr rymohr commented May 15, 2014

Rewrite https filter to replace http refs to :base_url instead of just http://github.com.

Resolves #123

@rymohr
Copy link
Contributor Author

rymohr commented May 19, 2014

The failed build is due to issues installing charlock_holmes on REE.

@jch
Copy link
Contributor

jch commented May 19, 2014

I don't see any dependencies to ruby versions in charlock_holmes in their gemspec https://github.com/brianmario/charlock_holmes/blob/master/charlock_holmes.gemspec. Was it a one-off error?

@rymohr
Copy link
Contributor Author

rymohr commented May 19, 2014

Is there a way to force travis to rerun the build?

@jch
Copy link
Contributor

jch commented May 21, 2014

@rymohr I think you might need to restart the build since it's building from your fork. http://stackoverflow.com/a/21727193/1309185 has instructions.

@rymohr
Copy link
Contributor Author

rymohr commented May 21, 2014

Yeah I probed around on the site but doesn't look like I have that option:

image

@simeonwillbanks
Copy link
Contributor

Closing in favor of #131

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.

De-github https filter
3 participants