You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Establish a file naming convention for golang files
Proposal
Currently the node-net-manager files do not follow a comprehensive file naming structure, which is desirable in any case.
As for golang, there is no documented styleguide for file naming, however general consensus seems to be all lower case, with snake case, where needed, while avoiding snake case suffixes like <file>_arm.go or <file>_linux.go, just like _test.go for tests. (see golang/go#36060)
My proposal is scanning the node-net-manager files and renaming long files to shorter ones where possible, while keeping the filenames as descriptive as possible. Generally a go-esque structure could be the following:
Short
Establish a file naming convention for golang files
Proposal
Currently the node-net-manager files do not follow a comprehensive file naming structure, which is desirable in any case.
As for golang, there is no documented styleguide for file naming, however general consensus seems to be all lower case, with snake case, where needed, while avoiding snake case suffixes like
<file>_arm.go
or<file>_linux.go
, just like_test.go
for tests. (see golang/go#36060)My proposal is scanning the
node-net-manager
files and renaming long files to shorter ones where possible, while keeping the filenames as descriptive as possible. Generally a go-esque structure could be the following:node-net-manager/
├─ environment/
├─ proxy/
│ ├─ tunnel.go
│ ├─ tunnel_test.go
│ ├─ cache.go
├─ mqtt/
│ ├─ tablequery.go
│ ├─ handlers.go
│ ├─ client.go
│ ├─ updates.go
├─ go.mod
├─ go.sum
├─ netmanager.go
Ratio
Better structure and a agreed upon file naming standard for this project
Impact
only file names in node-net-manager
Development time
1 hour
Status
looking for feedback
Checklist
The text was updated successfully, but these errors were encountered: