Skip to content

Conversation

@klauer
Copy link
Owner

@klauer klauer commented May 27, 2022

  • The types.EXP example may have been valid grammar back in the iec2xml days, but it does not appear to be valid for TwinCAT in 2022.
  • sample.EXP is simple enough and blark parses it well; added it to the test suite where it will be packaged/installed with blark

Partially addresses #21

The types.EXP may have been valid grammar back in the iec2xml days, but
it does not appear to be valid for TwinCAT in 2022.
@codecov-commenter
Copy link

codecov-commenter commented May 27, 2022

Codecov Report

❗ No coverage uploaded for pull request base (master@6044a3b). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #41   +/-   ##
=========================================
  Coverage          ?   67.20%           
=========================================
  Files             ?       16           
  Lines             ?     3687           
  Branches          ?        0           
=========================================
  Hits              ?     2478           
  Misses            ?     1209           
  Partials          ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6044a3b...cf2fca1. Read the comment docs.

@klauer
Copy link
Owner Author

klauer commented May 27, 2022

Hmm, looks like coverage support may still need some work. Reports are a good start, but the diff/comparison would be better.

Edit: oh, looks like I was just behind locally when making this branch. Will see what happens the next PR.

@klauer klauer merged commit 4ad8c51 into master May 27, 2022
@klauer klauer deleted the move_examples branch May 27, 2022 17:14
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.

3 participants