-
-
Notifications
You must be signed in to change notification settings - Fork 365
Add Two Bucket Test Generator #424
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
Comments
I'll deal with it. |
Question about file & class names. Exercise, project name and canonical-data are "TwoBucket". But class name in exercise is "TwoBuckets. As I see generator use class name also for finding canonical-data and here is the problem. The quickest option will be to change class name in exercise to TwoBucket? |
@GKotfis yep! Feel free to update the existing class. This case is specifically outlined in the provided generators documentation :) |
@GKotfis for some historical context, when this project started we didn't have any canonical data so each of the different tracks had a lot of liberty in their implementations (naming, what they tested, etc). Now we're trying to standardize to have a more consistent experience across tracks which is great, but we're trying to retrofit our existing solutions into the generators. For some exercises it's very straightforward but other exercises need a larger refactoring. |
Fixed by #447. |
This issue is part of an overall initiative to complete #195.
The documentation on how to add generators can be found here. If you get stuck or have any questions about adding this generator, please do not hesitate to reach out!
The text was updated successfully, but these errors were encountered: