bonsai/harakit
5
4
Fork 3
Code Issues 43 Pull Requests 3 Projects 1 Releases 36 Activity

Style guide #140

New Issue
Closed
opened 2024-07-10 00:32:48 -06:00 by emma · 3 comments
emma commented 2024-07-10 00:32:48 -06:00
Owner
Copy Link
Copy Source

The formatting of some tools in this repository is out-of-hand. @silt and I will be working on a STYLE file in the root of the tree.

The formatting of some tools in this repository is out-of-hand. @silt and I will be working on a STYLE file in the root of the tree.
emma added the enhancement label 2024-07-10 00:32:48 -06:00
emma self-assigned this 2024-07-10 00:32:48 -06:00
silt was assigned by emma 2024-07-10 00:32:48 -06:00
silt commented 2024-07-10 00:40:10 -06:00
Owner
Copy Link
Copy Source

embedded into the styleguide will be a matching .clang-format for the convenience of any interested parties

embedded into the styleguide will be a matching `.clang-format` for the convenience of any interested parties
emma pinned this 2024-07-15 14:18:32 -06:00
emma commented 2024-07-15 14:19:24 -06:00
Author
Owner
Copy Link
Copy Source

We should also embed a rustfmt.toml for Rust code.

We should also embed a `rustfmt.toml` for Rust code.
silt commented 2024-07-20 07:39:02 -06:00
Owner
Copy Link
Copy Source

unless someone else is interested in writing it, the .clang-format isn't going to happen. as far as i can tell, it's far too buggy to be even remotely useful. maybe it's a skill issue on my part, no clue. for future reference, here's what i've got:

Language: Cpp  # docs say to use Cpp for C as well
PointerAlignment: Left
ColumnLimit: 80
IndentWidth: 4
ContinuationIndentWidth: 4
UseTab: Always
LineEnding: LF
InsertBraces: true
BreakBeforeBraces: Attach
BreakBeforeBinaryOperators: All
AllowShortFunctionsOnASingleLine: Empty
AllowShortCaseLabelsOnASingleLine: false
AllowShortIfStatementsOnASingleLine: true
AllowShortLoopsOnASingleLine: true
AlignOperands: Align
AlignAfterOpenBracket: BlockIndent
AllowAllArgumentsOnNextLine: false
IndentCaseLabels: true
IndentCaseBlocks: true
IndentGotoLabels: true
SpaceBeforeCaseColon: false
SpaceBeforeParens: ControlStatements
SpacesBeforeTrailingComments: 2
SpaceBeforeAssignmentOperators: true
InsertNewlineAtEOF: true
unless someone else is interested in writing it, the `.clang-format` isn't going to happen. as far as i can tell, it's far too buggy to be even remotely useful. maybe it's a skill issue on my part, no clue. for future reference, here's what i've got: ```yaml Language: Cpp # docs say to use Cpp for C as well PointerAlignment: Left ColumnLimit: 80 IndentWidth: 4 ContinuationIndentWidth: 4 UseTab: Always LineEnding: LF InsertBraces: true BreakBeforeBraces: Attach BreakBeforeBinaryOperators: All AllowShortFunctionsOnASingleLine: Empty AllowShortCaseLabelsOnASingleLine: false AllowShortIfStatementsOnASingleLine: true AllowShortLoopsOnASingleLine: true AlignOperands: Align AlignAfterOpenBracket: BlockIndent AllowAllArgumentsOnNextLine: false IndentCaseLabels: true IndentCaseBlocks: true IndentGotoLabels: true SpaceBeforeCaseColon: false SpaceBeforeParens: ControlStatements SpacesBeforeTrailingComments: 2 SpaceBeforeAssignmentOperators: true InsertNewlineAtEOF: true ```
emma closed this issue 2024-07-29 22:59:57 -06:00
emma unpinned this 2024-07-29 23:00:00 -06:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 joke
you are silly!
 question
More information is needed
 wontfix
This won't be fixed
No labels enhancement
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
emma (Emma Tebibyte) nitori silt (silt!) soup (Soup For My Family) trinity (dtb)
No Assignees emma silt
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bonsai/harakit#140