-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Clarifies an error message in Katas to explain what is actually wrong. #11705
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
Conversation
|
@henryken @pabloem @aaltay - do you know how frequently this course is uploaded to https://stepik.org/? Does this happen for every release? |
|
@tvalentyn, we update the course when there are changes. |
|
Np! should we merge or close this PR? |
|
Run Website_Stage_GCS PreCommit |
|
I'm running the website staging to produce an archived version of the current website. Please disregard that : ) - I just looked for a recent PR. |
|
@henryken WDYT? It might make sense to merge this at least until you actually remove the check? |
|
Merging sounds good. What is https://stepik.org/ ? |
|
retest this please |
|
stepik is the platform where these courses are uploaded , an downloaded by intellij |
Thank you! :) |
|
Btw, I've submitted a pull request to remove all the placeholder checks |
Spent some time figuring out why I am getting "Incorrect" mark for a correct submission that matches the solution. After debugging the internals it turned out that my answer was written outside of the answer prompts. New message should explain the user what is wrong.
Post-Commit Tests Status (on master branch)
Pre-Commit Tests Status (on master branch)
See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.