-
Notifications
You must be signed in to change notification settings - Fork 119
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
Shortcuts for buses #1397
Shortcuts for buses #1397
Conversation
a1fdc67
to
cb5daff
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great clear code 👍
One thing I find Reasonable is instead of renaming Sorry my delirium, there’s no such thing as Holes.List.groupBy
to Holes.List.groupByString
move it to Holes.List.String.groupBy
so that the Belt convention is keptBelt.List.String
. The code is OK
Did not hard-tested the functionality yet |
cb5daff
to
a6d32dc
Compare
One thing nice to have which was not described in the issue is some special terminal handling. Steps to reproduce
Desired behaviorThe bus gets label Actual behaviorThe bus gets label |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
a6d32dc
to
29cba2e
Compare
… return Project.t instead of Result
…): shortcuts for creating bus nodes Closes #1365
3a7eadb
to
0f97ec9
Compare
Closes #1365
Also see xodio/xod-docs#52