Skip to content

Fix - removed indentation in tutorial#1027

Merged
anthonygego merged 1 commit into
INGInious:masterfrom
anaHue:correct-doc
Feb 11, 2025
Merged

Fix - removed indentation in tutorial#1027
anthonygego merged 1 commit into
INGInious:masterfrom
anaHue:correct-doc

Conversation

@anaHue

@anaHue anaHue commented Dec 8, 2024

Copy link
Copy Markdown
Contributor

This is a PR to solve an issue related to the doc.

Looking at the tutorial of how to write an hello world exercise, it stated an example of the code to put in a template file to run the student code.

The problem is that the student code is inside the function and the funtion was called at the end of the function instead of being called after the function definition. This would result in not being able to run the student code but even by running the function it will be an infinite loop.

To solve this, I had to remove one indentation level as Python works with identation at the right level to call a function. By testing, the code was indeed showing nothing before the change and displayed "hello world" after changes.

@AlexandreDoneux

Copy link
Copy Markdown
Contributor

You gave no explanation on what the problem was and how you fixed it. Please provide those in your PR description.

Also provide proof there was a problem and that your changes are valid and do not cause more problems.

@anaHue

anaHue commented Dec 20, 2024

Copy link
Copy Markdown
Contributor Author

Indeed I forgot to give context as I thought it was clear of the changes I did. I added what you asked and hope it is greatly structured. Thanks for the feedback, I tend to forgot to bring context.

@nrybowski nrybowski force-pushed the master branch 2 times, most recently from e53b0e2 to 9a6d192 Compare December 21, 2024 13:51
@anthonygego anthonygego merged commit 54a65c9 into INGInious:master Feb 11, 2025
@anthonygego anthonygego deleted the correct-doc branch February 11, 2025 07:42
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.

3 participants