-
Notifications
You must be signed in to change notification settings - Fork 4
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
CI #16
Comments
@4meta5 we could add a step that expands the macros(but I think we will need tests or examples for this to be useful)
Aditionally we could |
Marking this as closed in favor of #18. If we find things we need to add specific to macros we can come back and reopen this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Ideally CI would verify that the macros output valid Rust code. Compiling the crate itself is not as helpful because it does not tell us if the macro expansion is valid Rust.
Note that this ideal may first require #8 because most of our macro output isn't valid Rust until deps include all that are required.
The text was updated successfully, but these errors were encountered: