sipeed / bl602-pac

Embedded Rust's Peripheral Access Crate for BL602 microcontrollers
MIT License
19 stars 10 forks source link

Add Github actions #7

Closed 9names closed 3 years ago

9names commented 3 years ago

Manually checking that SVD changes don't break PAC svd2rust generation is painful. Automate it using Github Actions so we can run against several Rust versions and have some confidence that PRs are sane

9names commented 3 years ago

I ran this CI against my fork. If you want to verify what a pass looks like: https://github.com/9names/bl602-pac/pull/1/checks

username223 commented 3 years ago

Might be a good idea to also run SVDConv to check that the SVD is semantically correct. Running it with only the SVD file name will check the file for errors (return code 2), warnings (return code 1) or nothing (return code 0). It also supports the (undocumented in --help) option --strict which turns some warnings into errors and adds 22 more warnings. The return code would have to be ignored until the existing warnings/errors have been fixed.

This might also be better done at the new SVD repo at Pine64.