Skip to content
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

Typo - Existant -> Existent #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Sardtok
Copy link

@Sardtok Sardtok commented Jul 26, 2014

Just a minor typo in the test class.

Just a minor typo in the test class.
@juleswhite
Copy link
Owner

I really appreciate you catching this.

I will definitely merge this change after the current course ends. The change affects the naming of tests cases and will require updates to our auto-grading service in the cloud (so that it generates the updated feedback messages) as well as everyone that has checked out the assignment. If we update now, there is the potential that folks may see a slight mismatch between their local feedback and the official feedback (I know it would be very minor) and get confused. It is easier to accept that I made a spelling mistake and live with it until the end of the course than to roll out a change and potentially cause confusion.

@Sardtok
Copy link
Author

Sardtok commented Jul 28, 2014

No problem. I figured the rubric annotations might make it a problem to merge, but thought maybe it was early enough still.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants