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 modules required for build to README #4

Merged
merged 1 commit into from Jan 29, 2015

Conversation

Projects
None yet
2 participants
@pwr22

pwr22 commented Jan 28, 2015

Also rebuilds README pulling in twitter_args

@pwr22

This comment has been minimized.

Show comment
Hide comment
@pwr22

pwr22 Jan 29, 2015

A bit more info on this. As someone who's never used module::install I had to work this out before I could get on with contributing anything. So I added this new section to remove that barrier for anyone else in the future

pwr22 commented Jan 29, 2015

A bit more info on this. As someone who's never used module::install I had to work this out before I could get on with contributing anything. So I added this new section to remove that barrier for anyone else in the future

@semifor

This comment has been minimized.

Show comment
Hide comment
@semifor

semifor Jan 29, 2015

Owner

I haven't merged this one yet, because I want to take a look when I have a bit of time. I'm not sure this is strictly correct. Rather than BUILDING, perhaps CONTRIBUTING. I don't think you need the additional modules to build twirc if you install it from CPAN. But you do if you're working with the github repo. I need to verify.

Owner

semifor commented Jan 29, 2015

I haven't merged this one yet, because I want to take a look when I have a bit of time. I'm not sure this is strictly correct. Rather than BUILDING, perhaps CONTRIBUTING. I don't think you need the additional modules to build twirc if you install it from CPAN. But you do if you're working with the github repo. I need to verify.

@pwr22

This comment has been minimized.

Show comment
Hide comment
@pwr22

pwr22 Jan 29, 2015

Ah yes, I mean building the distribution, I'll make it clearer

Peter
On 29 Jan 2015 02:53, "Marc Mims" notifications@github.com wrote:

I haven't merged this one yet, because I want to take a look when I have a
bit of time. I'm not sure this is strictly correct. Rather than BUILDING,
perhaps CONTRIBUTING. I don't think you need the additional modules to
build twirc if you install it from CPAN. But you do if you're working with
the github repo. I need to verify.


Reply to this email directly or view it on GitHub
#4 (comment).

pwr22 commented Jan 29, 2015

Ah yes, I mean building the distribution, I'll make it clearer

Peter
On 29 Jan 2015 02:53, "Marc Mims" notifications@github.com wrote:

I haven't merged this one yet, because I want to take a look when I have a
bit of time. I'm not sure this is strictly correct. Rather than BUILDING,
perhaps CONTRIBUTING. I don't think you need the additional modules to
build twirc if you install it from CPAN. But you do if you're working with
the github repo. I need to verify.


Reply to this email directly or view it on GitHub
#4 (comment).

Add modules required for build to README
Also rebuilds README pulling in twitter_args
@pwr22

This comment has been minimized.

Show comment
Hide comment
@pwr22

pwr22 Jan 29, 2015

I've updated the docs again, hopefully this makes it clearer

pwr22 commented Jan 29, 2015

I've updated the docs again, hopefully this makes it clearer

@semifor

This comment has been minimized.

Show comment
Hide comment
@semifor

semifor Jan 29, 2015

Owner

Thanks! That works!

Owner

semifor commented Jan 29, 2015

Thanks! That works!

semifor pushed a commit that referenced this pull request Jan 29, 2015

Marc Mims
Merge pull request #4 from pwr22/build-docs
Add modules required for build to README

@semifor semifor merged commit ea0d588 into semifor:master Jan 29, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment