Turn off ri and rdoc generation by default #42

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
@dhh

dhh commented Feb 25, 2011

Hi guys,

It seems like everyone and their mother has --no-ri and --no-rdoc in their .gemrc file. Any objections to pulling that convention upstream?

@pillowfactory

This comment has been minimized.

Show comment
Hide comment
@nathanhoad

This comment has been minimized.

Show comment
Hide comment

+1

@bjreath

This comment has been minimized.

Show comment
Hide comment

bjreath commented Feb 25, 2011

+1

@vinayvinay

This comment has been minimized.

Show comment
Hide comment

+1

@febuiles

This comment has been minimized.

Show comment
Hide comment

+1

@ismasan

This comment has been minimized.

Show comment
Hide comment

ismasan commented Feb 25, 2011

+1

@jeremyherbert

This comment has been minimized.

Show comment
Hide comment
@jeremyherbert

jeremyherbert Feb 25, 2011

+1 please dear world please

+1 please dear world please

@blahed

This comment has been minimized.

Show comment
Hide comment

blahed commented Feb 25, 2011

+1

@redinger

This comment has been minimized.

Show comment
Hide comment

+1

@jaredbrown

This comment has been minimized.

Show comment
Hide comment

+1

@Yardboy

This comment has been minimized.

Show comment
Hide comment

Yardboy commented Feb 25, 2011

+1

@jm

This comment has been minimized.

Show comment
Hide comment
@jm

jm Feb 25, 2011

+1 definitely.

jm commented Feb 25, 2011

+1 definitely.

@ethangunderson

This comment has been minimized.

Show comment
Hide comment
@mariovisic

This comment has been minimized.

Show comment
Hide comment
@mariovisic

mariovisic Feb 25, 2011

+1 ri and rdoc are available online usually so it's only useful for offline coding (in which case you could generate your own documentation very easily).

+1 ri and rdoc are available online usually so it's only useful for offline coding (in which case you could generate your own documentation very easily).

@kouky

This comment has been minimized.

Show comment
Hide comment

kouky commented Feb 25, 2011

+1

@darrenterhune

This comment has been minimized.

Show comment
Hide comment
@darrenterhune

darrenterhune Feb 25, 2011

+ten hundred million thousand... I win

+ten hundred million thousand... I win

@luke0x

This comment has been minimized.

Show comment
Hide comment
@luke0x

luke0x Feb 25, 2011

Would --no-ri be needed in install_update_defaults_str too?

luke0x commented on 8c70f33 Feb 25, 2011

Would --no-ri be needed in install_update_defaults_str too?

This comment has been minimized.

Show comment
Hide comment

+1

@paulodeleo

This comment has been minimized.

Show comment
Hide comment

+1

@vanntastic

This comment has been minimized.

Show comment
Hide comment

+1

@campreb

This comment has been minimized.

Show comment
Hide comment

campreb commented Feb 25, 2011

+1

@rbazinet

This comment has been minimized.

Show comment
Hide comment

+1

@jdotw

This comment has been minimized.

Show comment
Hide comment

jdotw commented Feb 25, 2011

+1

@andybluntish

This comment has been minimized.

Show comment
Hide comment

+1

@cikenstrip

This comment has been minimized.

Show comment
Hide comment

+1

@radar

This comment has been minimized.

Show comment
Hide comment
@radar

radar Feb 25, 2011

Holy crap @ response.

+1

radar commented Feb 25, 2011

Holy crap @ response.

+1

@radar

This comment has been minimized.

Show comment
Hide comment
@radar

radar Feb 25, 2011

P.S. / FYI: my mother doesn't have this option turned on

radar commented Feb 25, 2011

P.S. / FYI: my mother doesn't have this option turned on

@jackhq

This comment has been minimized.

Show comment
Hide comment

jackhq commented Feb 25, 2011

+1

@plukevdh

This comment has been minimized.

Show comment
Hide comment

+1

@lewisou

This comment has been minimized.

Show comment
Hide comment

lewisou commented Feb 25, 2011

+1

@trevrosen

This comment has been minimized.

Show comment
Hide comment

+1

@cclow

This comment has been minimized.

Show comment
Hide comment

cclow commented Feb 25, 2011

+1

@joeyw

This comment has been minimized.

Show comment
Hide comment

joeyw commented Feb 25, 2011

+1

@ccollins

This comment has been minimized.

Show comment
Hide comment

+1

@morexchange

This comment has been minimized.

Show comment
Hide comment

+1

@benhoskings

This comment has been minimized.

Show comment
Hide comment

+1

@geetarista

This comment has been minimized.

Show comment
Hide comment

+1

@fjoachim

This comment has been minimized.

Show comment
Hide comment

+1

@armstrjare

This comment has been minimized.

Show comment
Hide comment

+1

@naudo

This comment has been minimized.

Show comment
Hide comment

naudo commented Feb 25, 2011

+1

@parndt

This comment has been minimized.

Show comment
Hide comment
@parndt

parndt Mar 2, 2011

Contributor

+1 justinbaker

Couldn't resist.

Honestly, It's a huge shame that this pull request has been closed as it offers one of the most significant performance increases to ruby developers who just want to get on with the job and install lots of gems without this massive overhead every. single. time.

I always thought rubygems was a tool by the ruby community for the ruby community but it seems the community has spoken and been swiftly ignored. "No correspondence will be entered into" I guess?

Contributor

parndt commented Mar 2, 2011

+1 justinbaker

Couldn't resist.

Honestly, It's a huge shame that this pull request has been closed as it offers one of the most significant performance increases to ruby developers who just want to get on with the job and install lots of gems without this massive overhead every. single. time.

I always thought rubygems was a tool by the ruby community for the ruby community but it seems the community has spoken and been swiftly ignored. "No correspondence will be entered into" I guess?

@apneadiving

This comment has been minimized.

Show comment
Hide comment

+1

@scotttam

This comment has been minimized.

Show comment
Hide comment

scotttam commented Mar 3, 2011

+1

@mihael

This comment has been minimized.

Show comment
Hide comment

mihael commented Mar 3, 2011

+X

@peter

This comment has been minimized.

Show comment
Hide comment

peter commented Mar 3, 2011

+1

@buffpojken

This comment has been minimized.

Show comment
Hide comment

+1

@asiazhang

This comment has been minimized.

Show comment
Hide comment

+1

@dstockdale

This comment has been minimized.

Show comment
Hide comment

+1

@jerome

This comment has been minimized.

Show comment
Hide comment

jerome commented Mar 3, 2011

+1

@hugobarauna

This comment has been minimized.

Show comment
Hide comment

+1

@hugobarauna

This comment has been minimized.

Show comment
Hide comment

+1

@tatygrassini

This comment has been minimized.

Show comment
Hide comment

+1

@williamn

This comment has been minimized.

Show comment
Hide comment

williamn commented Mar 4, 2011

+1

@wxianfeng

This comment has been minimized.

Show comment
Hide comment
@wxianfeng

wxianfeng Mar 4, 2011

+1
every time i am add --no-ri --no-rdoc

+1
every time i am add --no-ri --no-rdoc

@PanosJee

This comment has been minimized.

Show comment
Hide comment

PanosJee commented Mar 4, 2011

+1

@pklingem

This comment has been minimized.

Show comment
Hide comment

pklingem commented Mar 4, 2011

+1

@b-mandelbrot

This comment has been minimized.

Show comment
Hide comment

wait +1

@jmonteiro

This comment has been minimized.

Show comment
Hide comment

+1

@jjb

This comment has been minimized.

Show comment
Hide comment
@jjb

jjb Mar 9, 2011

Contributor

Hey folks. I made this compromise patch which educates users how to use rdoc and ri, and how to not install them in the future. check it out! #44

Contributor

jjb commented Mar 9, 2011

Hey folks. I made this compromise patch which educates users how to use rdoc and ri, and how to not install them in the future. check it out! #44

@burningTyger

This comment has been minimized.

Show comment
Hide comment

+1

@codatory

This comment has been minimized.

Show comment
Hide comment

+1

@rickthomasjr

This comment has been minimized.

Show comment
Hide comment

+1

@parndt

This comment has been minimized.

Show comment
Hide comment
@parndt

parndt Mar 23, 2011

Contributor

Good news, you can now unsubscribe to notifications from just this pull request. https://github.com/blog/821-mention-somebody-they-re-notified

Contributor

parndt commented Mar 23, 2011

Good news, you can now unsubscribe to notifications from just this pull request. https://github.com/blog/821-mention-somebody-they-re-notified

@asanghi

This comment has been minimized.

Show comment
Hide comment
@asanghi

asanghi Mar 27, 2011

This is pull number 42.

asanghi commented Mar 27, 2011

This is pull number 42.

@sawanoboly

This comment has been minimized.

Show comment
Hide comment

+1

@szymon-przybyl

This comment has been minimized.

Show comment
Hide comment
@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Aug 3, 2011

You'll have my vote as well.

+1

ghost commented Aug 3, 2011

You'll have my vote as well.

+1

@charliesome

This comment has been minimized.

Show comment
Hide comment
@charliesome

charliesome Oct 31, 2011

Contributor

+1

Contributor

charliesome commented Oct 31, 2011

+1

@celeduc

This comment has been minimized.

Show comment
Hide comment
@celeduc

celeduc Sep 14, 2012

+1, now more than ever.

celeduc commented Sep 14, 2012

+1, now more than ever.

@xfstart07

This comment has been minimized.

Show comment
Hide comment

+1

@garethrees

This comment has been minimized.

Show comment
Hide comment

+1

@PikachuEXE

This comment has been minimized.

Show comment
Hide comment
@PikachuEXE

PikachuEXE Mar 5, 2013

+1
How come this is not merged?

+1
How come this is not merged?

@lukaselmer

This comment has been minimized.

Show comment
Hide comment

+1

@araipiyo

This comment has been minimized.

Show comment
Hide comment

+1

@mkristian

This comment has been minimized.

Show comment
Hide comment

+1

@PofMagicfingers

This comment has been minimized.

Show comment
Hide comment
@PofMagicfingers

PofMagicfingers Oct 22, 2013

pull request closed ? but ... +1 !

pull request closed ? but ... +1 !

@PikachuEXE

This comment has been minimized.

Show comment
Hide comment
@PikachuEXE

PikachuEXE Oct 22, 2013

Why is it closed? Maybe GitHub bug or something
Anyway +1

Why is it closed? Maybe GitHub bug or something
Anyway +1

@lukaseder

This comment has been minimized.

Show comment
Hide comment
@lukaseder

lukaseder Jul 1, 2014

-1, actually

-1, actually

@mkristian

This comment has been minimized.

Show comment
Hide comment
@mkristian

mkristian Jul 2, 2014

very much +1
we use this default with jruby already ;)

very much +1
we use this default with jruby already ;)

@sagarey

This comment has been minimized.

Show comment
Hide comment

sagarey commented Jan 19, 2015

+1

This issue was closed.

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