-
Notifications
You must be signed in to change notification settings - Fork 44
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
[Rule Guideline] Modify Rule 17 to account for same subdirectories in submissions #40
Comments
Assign please. I already have the mkiocccentry code updated. Now I am waiting on the function in jparse and then I can finish txzchk. I really will be going afk now! I have my right leg dangling out of my bed so that I have to. Back later ... please do a pull request in jparse with that function you refer to as you have the idea in your head and I am not clear entirely what it should do. Thanks! |
P.S. In regards to @SirWumpus' idea of supplying a test suite with lots of files Such a tarball would be untarred some Makefile rule (say See GH-issuecomment-2567936096 in "the other repo". |
Status updateThe code needed to implement this enhancement is being worked on in "the other repo" issue 1060. Meanwhile we plan to modify the rules and update the guidelines and likely the FAQ to accompany the work being done in "the other repo" in the near future. So in this case the rules and guidelines will likely lead the work needed to implement. I.e., tools such as We are going to lead with modifying the rules and guidelines so that it becomes more clear what the change will become to the developers in "the other repo". UPDATE 0Stay tuned .. P.s., Our goal is to have this issue, and the "the other repo" issue 1060 resolved and tested by mid-January, well ahead of the date when the contest is anticipated to open. |
If you forgot .. there's actually a test suite. It was built in long ago. It does not need to be a tarball per se as there's the read as text file mode a part of the test suite. There are already lots of test files and indeed more would be added when this is done. |
UPDATE 0In fact it was one of the first things I did - so I could make sure it works okay (that and using the tarballs you provided). |
We believe we have addressed all of the current questions that still need answering at this time. If we've missed something or something else needs to be clarified, please ask again. |
Is there an existing issue for this?
Which Rule and/or Guideline has a problem?
Rule 17.
As well as:
What is wrong with the Rule and/or Guideline?
Rule 17 disallows sane subdirectories in submissions.
How do you recommend we fix the problem?
Once issue 1060 in the mkiocccentry repo is resolved, then Rule 17 should be modified accordingly.
Anything else?
No response
The text was updated successfully, but these errors were encountered: