-
Notifications
You must be signed in to change notification settings - Fork 440
Rename children that refer to a MemberDeclBlock
from members
to memberBlock
#1524
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
Conversation
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.
I definitely prefer memberBlock
. Not sure about members
-> items
, I slightly prefer members
I think. But they're both almost synonyms so 🤷
00c7cfb
to
575c9f8
Compare
MemberDeclBlock
from members
to memberBlock
and MemberDeclBlock.members
to items
MemberDeclBlock
from members
to memberBlock
swiftlang/swift#65126 @swift-ci Please test Windows |
…memberBlock` Previously, we frequently had patterns where we did `node.members.members`, which doesn’t read very nice. `node.memberBlock.members` is much nicer.
575c9f8
to
5ba5cd3
Compare
swiftlang/swift#65126 @swift-ci Please test Windows |
4 similar comments
swiftlang/swift#65126 @swift-ci Please test Windows |
swiftlang/swift#65126 @swift-ci Please test Windows |
swiftlang/swift#65126 @swift-ci Please test Windows |
swiftlang/swift#65126 @swift-ci Please test Windows |
Rename children that refer to a `MemberDeclBlock` from `members` to `memberBlock`
Rename children that refer to a `MemberDeclBlock` from `members` to `memberBlock`
Previously, we frequently had patterns where we did
node.members.members
, which doesn’t read very nice.node.memberBlock.members
is much nicer.