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

Website #199

Merged
merged 6 commits into from
Sep 17, 2018
Merged

Website #199

merged 6 commits into from
Sep 17, 2018

Conversation

maelle
Copy link
Member

@maelle maelle commented Sep 17, 2018

cf #193 #197

Not sure the sections are the best, but at least the coercions are at the bottom now.

There's a "queries" tag for spocc_objects but I had to insert its name in the config file as well, not sure why, wasn't able to reproduce this bug on a minimal package. Note that in the current version of the website, spocc_objects is absent from the reference index see https://ropensci.github.io/spocc/reference/index.html (but https://ropensci.github.io/spocc/reference/spocc_objects.html exists!)

@maelle maelle requested a review from sckott September 17, 2018 08:53
@maelle
Copy link
Member Author

maelle commented Sep 17, 2018

I also used this PR as an opportunity to add rOpenSci as a funder in DESCRIPTION, with the logo in the pkgdown website.

@maelle
Copy link
Member Author

maelle commented Sep 17, 2018

The Travis build fails but it fails for the master branch too. 🤔

@sckott
Copy link
Contributor

sckott commented Sep 17, 2018

there's a failing example I'll take care of later, and some tests that often fail only on travis that I haven't figured out yet

@sckott
Copy link
Contributor

sckott commented Sep 17, 2018

thanks @maelle

@sckott sckott added the docs label Sep 17, 2018
@sckott sckott added this to the v0.9 milestone Sep 17, 2018
@sckott sckott merged commit a00d595 into master Sep 17, 2018
This was referenced Sep 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants