-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update Introduction vignette to highlight new features #59
Comments
Perhaps examples should be added to the functions as well but this can also be done after the 1.0.0 move |
Will be following this guide (https://combine-australia.github.io/r-pkg-dev/versioning.html) for versioning in the future as I haven't been super consistent with it. Slightly irrelevant but it would be fun to release the 1.0.0 on June 21st (the 1 year anniversary of the package) if you're ok with that. |
I like it. Package looks great for 1 year! Thoughts on submitting to CRAN once we hit stable 1.0 and have examples added? |
Would love to submit to CRAN. I'm a bit nervous about the process but I think we're well on our way |
Worked on by #108 |
What else should we do before CRAN? @pbulsink |
Likely finish #108, come to a conclusion on #78 (do we need to do multi-driver function or just let the users do it?), look at #113, ensure our version number is correct, and then maybe go for it? If CRAN have additional suggestions after our first submit attempt we can have a better idea of what's needed. Once submitted we'll have to do PRs to a dev/beta branch because CRAN would prefer updates only happen every few months. |
Also working in pbulsink/f1dataR:checks-before-cran, so wait for a pr from there? |
As an alternative to implementing multi-driver #78 we could better
highlight (or reference in the function docs) loading 2+ drivers to compare
their telemetry in the plot in the Introduction vignette.
…On Sun, Jul 9, 2023, 16:51 Santiago Casanova ***@***.***> wrote:
Yes maybe we can hold off on #78
<#78> as it's not necessarily
essential. I'm with you on #113
<#113>
—
Reply to this email directly, view it on GitHub
<#59 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABJLHRSDVNI7XDGBZMJEJ43XPMKTPANCNFSM6AAAAAAY7WV4MM>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I have planned to do this for a while but especially now that the session laps function has been added. Also inclusion of other vignettes
The text was updated successfully, but these errors were encountered: