use fictional names instead of folder paths #113
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rongxin-liu
requested changes
Jul 24, 2025
Contributor
rongxin-liu
left a comment
rongxin-liu
left a comment
There was a problem hiding this comment.
Would there be a way we can still show the original folder paths somewhere in the UI, so the grader can find out the actual submitter?
Maybe somewhere in this UI or another place to show it? We can also render it in smaller text with a dimmer color, so that it is subtle. At the end of the day, we still need to locate the actual submitter if we find some similar code.
Contributor
Author
|
Thanks for the suggestion @rongxin-liu! I just pushed a commit, so now if you hover over the fictional name, the original submission path should be visible. |
rongxin-liu
approved these changes
Jul 25, 2025
Contributor
|
Looks good! Merging! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fixes issue #89