-
Notifications
You must be signed in to change notification settings - Fork 73
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
Print list #234
Print list #234
Conversation
Codecov Report
@@ Coverage Diff @@
## master #234 +/- ##
==========================================
+ Coverage 99.06% 99.21% +0.14%
==========================================
Files 78 79 +1
Lines 639 633 -6
Branches 106 106
==========================================
- Hits 633 628 -5
+ Misses 6 5 -1
Continue to review full report at Codecov.
|
I think I like this. Most usages use default arguments, so that's nice. The only problem I see is that when explicit arguments are added, it gets a little cryptic. What do you think about having just an
|
done, this idea was on the back of my mind. Glad that you proposed it aswell. |
this is really cool, let's merge and deploy a new version |
just playing with the idea of having a
printList
function (provided by @fvictorio) for most of our splitting and indenting cases.The function just needs an array, but can also receive a
firstSeparator
(usually asoftline
used to separate from the previous character), a normalseparator
to put between items (usually aconcat([',', line])
), and finally thelastSeparator
that in most cases is the same asfirstSeparator
with 2 exceptions (AssemblySwitch
andStructDefinition
).Please have a look and see if it makes sense to use this pattern.