-
Notifications
You must be signed in to change notification settings - Fork 21
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
Review scoringutils 2.0.0 #373
Comments
Hi @nikosbosse, yes, I can do this 👍 |
I think we are getting close to this being done. Should we set a date for this that works for everyone? |
Hi all. We are really very close to this now! @jamesmbaazam @Bisaloo would you have time you do this in the next few weeks? As a reminder the idea is to use the Ropensci review template and do a pseudo submission review. Ideally issues would be split into really needs to be done now and scored with some kind of urgency (so we can slot them into future releases in a way that makes sense and doesn't drive Nikos to post too many memes). |
Ping!! |
I can do this over the next two weeks if that works for you. |
That would be really great! |
For awareness when looking these are the outstanding issues: https://github.com/epiforecasts/scoringutils/issues?q=is:issue%20is:open%20sort:updated-desc%20project:epiforecasts/6 |
Can I follow the steps documented here: https://epiverse-trace.github.io/blueprints/code-review.html#full-package-review to open a PR with the full codebase as diff, so I can comment directly on specific portions of the code using GitHub PR review infrastructure? |
Once the major update that's currently in progress is done it would be great to have another package review like the ones done in #121 and #179.
@Bisaloo would you be up for another review once the update is finished?
The text was updated successfully, but these errors were encountered: