Skip to content

Conversation

@codingthat
Copy link
Contributor

No description provided.

@meatball133 meatball133 added the x:rep/small Small amount of reputation label Apr 20, 2025
@meatball133
Copy link
Member

meatball133 commented Apr 20, 2025

A thing I usually try to do is that if I sync the metadata (the toml file), if there are any changes so do I also update the test file and if I don't want to update the test file, I don't update the metadata. This at least to me makes it a lot easier to track what is synced with problem specs and what is not. Some tracks have just synced metadata without updating the test files and then is it very difficult to go in afterhand to know what has to be updated.

This can be removed by having an automated system which some tracks have, but it is a decision you have to make for yourself if it is worth implementing.

In this case the test changes is only involving changes in the test names, which doesn't really matter (according to myself). So you if you want I can merge these changes.

@codingthat
Copy link
Contributor Author

@meatball133 very true! Sorry I overlooked that here, makes sense. Thanks!

Not sure if I'll go full automation yet...depends how many exercises we want to add in the end. But anyway, the tests now match. ✔

@meatball133 meatball133 merged commit d7aa402 into exercism:main Apr 21, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:rep/small Small amount of reputation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants