bonsai/harakit
5
4
Fork 3
Code Issues 37 Pull Requests 7 Projects 1 Releases 30 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

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

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

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

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 Label
bug
duplicate
enhancement
help wanted
invalid
joke
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
emma
silt
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: bonsai/harakit#140
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?