OpenZeppelin / polkadot-runtime-wrappers

GNU General Public License v3.0
0 stars 1 forks source link

CI #16

Open 4meta5 opened 2 weeks ago

4meta5 commented 2 weeks ago

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.

ggonzalez94 commented 2 days ago

@4meta5 we could add a step that expands the macros(but I think we will need tests or examples for this to be useful)

- name: Expand macros
      run: cargo expand 

Aditionally we could trybuild for testing WDYT?

ggonzalez94 commented 2 days ago

I created #18 since I wasn't sure what to add for verifying the macro itself, but if we are happy with this we can close the issues with #19