Open Lawrence37 opened 4 years ago
See branch autograde-all. @aalmanza1998 Should we open a PR?
Should we try to get it merged to upstream nbgrader?
It doesn't have to be merged. We can just open a PR and let the community decide what to do with it. Also, I wrote this:
We have a button which auto-grades all submissions and a couple of JupyterLab extensions. We'll open pull requests for them once they are ready.
I am obligated to open a PR if the button is ready.
Ok, that sounds good to me. We can try opening a PR.
We need to update the branch though if we are opening a PR with upstream nbgrader. We still have it forked from lzach so there's been some changes in nbgrader not reflected on our branch/fork.
Actually, we don't have tests and documentation. No pull request at this time.
In terms of documentation I don’t think we need much besides maybe adding something here https://nbgrader.readthedocs.io/en/stable/user_guide/managing_assignment_files.html
Or what do you think?
Yes, not much documentation is necessary.
Move the commits to another branch and open a pull request to jupyter/nbgrader.