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 INSTALL.md - fixes #152 #155

Closed
wants to merge 1 commit into from
Closed

Update INSTALL.md - fixes #152 #155

wants to merge 1 commit into from

Conversation

absorber
Copy link

Added documentation about perl dependencies to reflect this issue - #152

Added documentation about perl dependencies to reflect this issue - #152
@ncanceill
Copy link

Excellent, now I can send a PR to your own branch with my own small changes to what you changed.

@ncanceill
Copy link

Oh, one last thing, when I asked you to add "fixes" to the PR description, I was not talking about the PR title, but the description text (your first comment).

@RichiH
Copy link
Owner

RichiH commented Jun 8, 2015

Can you rewrite this to state that you only need ronn in master, not release?

#167 should also make prove opportunistic.

I can easily rewrite it myself, but I guess you want the honour of the patch :)

@RichiH RichiH closed this Mar 29, 2021
@RichiH RichiH deleted the branch RichiH:master March 29, 2021 14:44
@alerque
Copy link
Collaborator

alerque commented Apr 2, 2021

@RichiH This was another of the automatic GitHub closures, it looks like the same reason applies (upstream fork removed). I would re-submit it, but I think this one is obsolete. The things it adds are no longer required for install, only for test, and that is properly document later in the file.

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.

4 participants