Sync triangle and add stub#537
Conversation
|
This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested. If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos. For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping |
|
|
||
| implementation | ||
| // Provide your implementation here | ||
| end. No newline at end of file |
There was a problem hiding this comment.
| end. | |
| end. | |
Last line is exceptional with a missing EOL.
There was a problem hiding this comment.
Other files in this repo don’t end with a blank line though so the file is fine as written.
There was a problem hiding this comment.
I have not been vigilant enough, then. Just because the others are wrong, doesn't mean that we should not fix that. ;)
There was a problem hiding this comment.
This is a text file, as such it should be consistent with its termination of lines. Otherise I have to check and fix it when I concatenate information later. (Headings end up being trailing the last "line" of a file when the last "line" ends with EOL plus more characters.) Or I have to check on the rendering for something I really should not have to check for.
|
Embarcadero is now coming after me for a licensing issue related to my Exercism work. Therefore, this is my final contribution involving Delphi Pascal code. |
|
Licensing issue related to Exercism? |
|
Indirectly. It’s related to a Delphi 11 Community Edition license I obtained two years ago when I was first taking a look at Delphi Pascal track here. |
|
@BNAndras unless you are profiting by the CE license I don't see what the issue would be as it relates to Exercism and the Delphi track specifically. At any rate, thanks for the work you have put in. |
This syncs the docs, metadata, and tests for
triangle. This adds two tests for scalene triangles that check other combinations of the two sides being equal. I also added an non-compiling stub that gives the student the minimum interface needed to pass the tests.After this, there are 37 exercises where
configletis reporting one or more tests missing since the test.toml files were last updated.