Skip to content

Conversation

@IsaacG
Copy link
Member

@IsaacG IsaacG commented Mar 24, 2025

No description provided.

@IsaacG IsaacG requested a review from vaeng March 24, 2025 19:21
@github-actions
Copy link
Contributor

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.

[no important files changed]

For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping @exercism/maintainers-admin in a comment. Thank you!

@vaeng
Copy link
Contributor

vaeng commented Mar 24, 2025

Hi Isaac, thanks for the work.

I guess this was automated? If yes, i bet you would love to make your script work for the structure with UUIDs, so it is easier in the future to track changes?

https://github.com/exercism/cpp/wiki/Test-Cases-for-Practice-Exercises

@IsaacG
Copy link
Member Author

IsaacG commented Mar 24, 2025

I guess this was automated? If yes, i bet you would love to make your script work for the structure with UUIDs, so it is easier in the future to track changes?

This was done by hand with the aid of configlet sync --update --tests. I looked at the output, hand edited the tests, and accepted the change.

@IsaacG
Copy link
Member Author

IsaacG commented Mar 24, 2025

I haven't written C++ in over a decade. If there is an existing test-generator tool that is used in some exercises, I could take a stab at applying it to other exercises. If there isn't, I can't take on creating one at this point.

Copy link
Contributor

@vaeng vaeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR. I added the uuids.

@vaeng vaeng merged commit 9067706 into exercism:main May 13, 2025
8 checks passed
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