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

Ready for CRAN? #38

Open
MatthewBJane opened this issue Sep 22, 2023 · 9 comments
Open

Ready for CRAN? #38

MatthewBJane opened this issue Sep 22, 2023 · 9 comments

Comments

@MatthewBJane
Copy link
Owner

Hey just wanted to see if we are ready for submitting this to CRAN yet? Is the only thing left including some tests? @christopherkenny mentioned he would add those so if that is all we need then that's awesome! Take your time Chris, I'm just curious if there is anything else I could do to get this CRAN ready

@christopherkenny
Copy link
Collaborator

Yeah, I think the only thing we really need now is some tests. The existing checks seem to run well and everything is structured fine, so shouldn't be a painful submission.

@christopherkenny
Copy link
Collaborator

Tests are added. One which runs the theme code and one which actually produces a plot if vdiffr is available. They're pretty simple but shouldn't be too restrictive.

@MatthewBJane
Copy link
Owner Author

Awesome thanks a ton Chris! Okay, @christopherkenny and @lukepilling I just followed some online instructions and now the package is submitted to CRAN!!

@lcpilling
Copy link
Collaborator

Amazing, well done guys. I've learned a lot about package development - thanks @christopherkenny

@MatthewBJane
Copy link
Owner Author

So recieved a reply back and it did not pass initial checks but I think it was the URL was wrong (it was matthewbjane.com/theme_park instead of matthewbjane.com/ThemePark). I re-submitted it

@christopherkenny
Copy link
Collaborator

Oops, forgot to check that. There's a little package urlchecker.

I suspect they may also be unhappy with the version number. We probably should have updated 0.0.0.9000 -> 0.0.1 or 0.1.0. Sorry forgot about checking the description details extra carefully for first submission.

@erikgahner
Copy link

First, thanks for a great package! A lot of interesting and fun themes.

Is there any update on this? I was looking for the package on CRAN to no avail.

@christopherkenny
Copy link
Collaborator

Good to go by me. Just fixed a small docs thing with the Elf theme.

@MatthewBJane
Copy link
Owner Author

Great! I will attempt to submit it today!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants