Replies: 1 comment 3 replies
-
Overall I'm not very impressed with dart's unit testing convention. It may be so new to me that I just need time to adapt. |
Beta Was this translation helpful? Give feedback.
3 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
There are some differences in this port compared to the python source. To start this discussion, I'll note the major difference.
CLI args
Some of the CLI options are actually implemented as switches (AKA flags) where the dart arguments library automatically adds a inverse for a specified flag:
This enables the flag
While this can disable the flag
I have an idea about how to work with this in the action.yml for cpp-linter-action repo, but we may not actually use this experimental source at all.
Lines changes only feature
This dart port only focuses on the lines with actual additions. The python source originally focused on all lines in the diff, but because of some confusion, this led to a change in the default behavior.
I am storing the data needed for both approaches in dart, but I found it easier to only focus on lines with additions. Thus, there is some dead code here.
We can talk more about others as we go, but I just wanted a place where we could discuss. I expect this discussion will lead to improvements in the python source.
Beta Was this translation helpful? Give feedback.
All reactions