DOC: Link to dev version of contributing guide from README.md

[jamesmyatt]

README.md should link to the development version of the contributing guide, since the stable version may not match current practice for master. For example, see #24069 .

I'm not sure this PR is a good idea, but I thought it was easiest open it and get comments directly.

[codecov]

Codecov Report

Merging #24172 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #24172      +/-   ##
==========================================
+ Coverage   92.21%   92.21%   +<.01%     
==========================================
  Files         162      162              
  Lines       51723    51723              
==========================================
+ Hits        47694    47695       +1     
+ Misses       4029     4028       -1

Flag	Coverage Δ
#multiple	90.61% <ø> (ø)	⬆️
#single	43.03% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
pandas/util/testing.py	87.41% <0%> (-0.1%)	⬇️
pandas/io/json/json.py	93.09% <0%> (+0.47%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 949b148...b5460c3. Read the comment docs.

[codecov]

Codecov Report

Merging #24172 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #24172   +/-   ##
=======================================
  Coverage   92.21%   92.21%           
=======================================
  Files         162      162           
  Lines       51723    51723           
=======================================
  Hits        47694    47694           
  Misses       4029     4029

Flag	Coverage Δ
#multiple	90.61% <ø> (ø)	⬆️
#single	43.03% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
pandas/core/arrays/datetimes.py	98.56% <0%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 949b148...0109662. Read the comment docs.

[jreback]

this would be fine. @jorisvandenbossche @TomAugspurger

[TomAugspurger]

Yeah, that's probably best. That link does occasionally break if a build fails, but not too often.

[datapythonista]

lgtm

[jamesmyatt]

There are 3 other places in the repo that could be changed too:

• .github/CONTRIBUTING.md line 6 and line 22
• doc/README.rst line 1
• doc/source/overview.rst line 95

My inclination is to change the first, but leave the others alone so that internal references in docs are maintained. What do you think?

[jamesmyatt]

I'm also concerned about duplication between .github/CONTRIBUTING.md (which I've only just found) and the contributing section of README.md.

[jorisvandenbossche]

My inclination is to change the first,

I think you can change the first two (and the third could actually use sphinx syntax to make a link, so it always links to the same version)

[datapythonista]

@jorisvandenbossche are you ok with this now?

[TomAugspurger]

I think changing the link in overview.rst to be a Sphinx ref makes sense.

[jreback]

thanks @jamesmyatt

[TomAugspurger]

@jamesmyatt would you be interested in making another PR to update that last link in contributing.rst?

…

On Mon, Dec 17, 2018 at 6:56 AM Jeff Reback ***@***.***> wrote: thanks @jamesmyatt <https://github.com/jamesmyatt> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#24172 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABQHIkVMBsOh8ebyg4U4Tu3C6tKYEQFxks5u55R2gaJpZM4ZKOZH> .

[jamesmyatt]

@TomAugspurger , sure.