Skip to content
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

add boilerplate for tests + move template to a new file #108

Open
wants to merge 1 commit into
base: v0.3
Choose a base branch
from

Conversation

oriongonza
Copy link
Contributor

This slightly improves the boilerplate gen. I think that cargo-aoc users always want the two tests that come for free

@gobanos
Copy link
Owner

gobanos commented Dec 4, 2023

Hey, thanks for your contribution!
However the PR is not based on current state of the branch, and have a similar behavior to cargo-aoc/template/src/day.rs.tpl.
Would you rebase your PR, or close it if the changes are similar enough?

@oriongonza oriongonza force-pushed the ardi/improve-codegen branch from f86fe44 to 3a61524 Compare December 4, 2023 16:41
@oriongonza
Copy link
Contributor Author

Done, this adds a bit more boilerplate for the tests, which almost always have the same structure.

@oriongonza
Copy link
Contributor Author

I personally use a different template than what I provided here in this PR. It might be useful to provide a template at runtime.

cargo input -g --template src/codegen.rs.tpl

@oriongonza oriongonza force-pushed the ardi/improve-codegen branch from 3a61524 to 0cbe3b0 Compare December 5, 2023 03:57
@oriongonza oriongonza force-pushed the ardi/improve-codegen branch from 0cbe3b0 to a3c4cdd Compare December 5, 2023 03:59
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