Replies: 1 comment
-
Rename AA: AA is for disconnected. connection_failed (All use connection_failed except WebSocketClient uses connection_error) |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Similar networking APIs have been added with inconsistent names.
My reasoning for consistency (weighed against each other): Reduced confusion (what timing? differentiated verb), consistent implied details (started what? cognitive load), cleaner reusable code (can: n in variant group: n.enabled = true), ease of knowing more features (A == B, A or B needed?), reduced double negatives, and reduced typing.
The networking category (includes MultiplayerAPI, MultiplayerPeer, WebSocketServer, WebSocketClient) is full of API naming Outliers.
This Proposal is for the category to be consistent within itself, for the above reasons.
Beta Was this translation helpful? Give feedback.
All reactions