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 CI workflow to check CDDL syntax #15

Merged
merged 2 commits into from
Nov 28, 2022
Merged

Conversation

fxamacker
Copy link
Owner

@fxamacker fxamacker commented Nov 28, 2022

Add GitHub Actions (cddl.yml) to check CDDL syntax:

  • checkout repo
  • install cddl compiler using cargo
  • extract CDDL from CCF specifications document and print SHA256 of extracted CDDL
  • compile extracted CDDL file

The CCF specification document must have a pair of CDDL comments to mark the CDDL notation to be extracted and compiled:

;CDDL-BEGIN
... 
;CDDL-END 

Closes #14

Thanks @turbolent for suggesting (in PR #10) that we use CI to check CDDL with https://github.com/anweiss/cddl.

@fxamacker fxamacker merged commit f5d85e6 into main Nov 28, 2022
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.

Add CI to extract CDDL from specs and validate it
1 participant