Update README.md with a link to Colin's blog post

sonatype-nexus-community / oysteR

Create purls from the filtered sands of your dependencies, powered by OSS Index

https://sonatype-nexus-community.github.io/oysteR/

Apache License 2.0

40 stars 9 forks source link

Update README.md with a link to Colin's blog post #26

Closed brittanybelle closed 4 years ago

brittanybelle commented 4 years ago

Colin recently wrote an outstanding blog post that would be especially helpful to new users of oysteR - let's promote it directly in the README!

cc @bhamail / @DarthHater / @csgillespie

csgillespie commented 4 years ago

@brittanybelle Made a few tweaks to the PR

Remember, you need to update the Rmd file not the md file ;)
Moved the logo. Avoids the need for links outside of the domain.
Resized the logo

brittanybelle commented 4 years ago

Thanks a bunch, @csgillespie ! Haven't returned to the R space in a few weeks so I totally forgot about .Rmd :) Thanks for the other details you added, too.

I noticed the logo you updated was pointing to tidyverse and left a couple comments including a suggested change, would appreciate your eyes on those as a sanity check when you have a moment (Jeffry is out of office for the next couple of weeks). Thank you!

csgillespie commented 4 years ago

@brittanybelle

I've updated the URL to point to this repo
I've moved the logo from docs/ to man/ This means that the logo will now "come with the package". It's a semi-standard R thing
Updated the Rmd and regenerated the .md file

bhamail commented 4 years ago

FWIW, I just reran the CI build and it was fine. Looks like an intermittent issue. I'll merge this in now the CI is happy.

brittanybelle commented 4 years ago

Ack, thanks for merging this @bhamail ! I totally lost the plot over the long weekend.

Thanks a bunch for polishing this up, @csgillespie !