-
Notifications
You must be signed in to change notification settings - Fork 7
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
Submit to multiple problems at once #2
base: master
Are you sure you want to change the base?
Conversation
let PROGRAM_VERSION be a string
marmoset -u xj2yu -c cs246
@@ -88,17 +88,35 @@ class MarmosetClient | |||
problem_links = {} | |||
|
|||
@course_page.links.find_all{|l| l.href.include? 'submitProject.jsp'}.each do |view_link| | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why the added whitespace?
Cool - I haven't touched this code in almost 4 years :) Thanks for the PR. In my opinion, this is really surprising default behaviour. Instead of making |
Thanks for creating this tool in the first place. What's amazing is that the Marmoset submission system is still being used :p I get that it would be more consistent to add a new flag for multiple submissions. However explicitly not typing in the problem id and filename would signify that the user doesn't just want to submit to a single problem. As far as I know Marmoset doesn't have any penalties against too many submissions, so it is not a risky thing to submit multiple projects by accident. I'm still making some changes so let me know if there are any other changes you want to discuss. |
"Explicitly not" is implicit :P I think it's pretty likely that someone would do it by accident. I forget to provide flags all the time, or accidentally hit enter early when typing commands. |
For some assignments, there are many parts to a single question but each file must be submitted on its own.
This feature will automatically submit files to matching problems.