Skip to content

feat: add clang format file - #79

Closed
Karilla wants to merge 3 commits into
smartobjectoriented:mainfrom
Karilla:add-clang-format
Closed

feat: add clang format file#79
Karilla wants to merge 3 commits into
smartobjectoriented:mainfrom
Karilla:add-clang-format

Conversation

@Karilla

Copy link
Copy Markdown
Contributor

As disscussed in #76 I have added a .clang-format file to simplify adherence to the coding conventions. For now, you need to manually enable clang-format in your code editor. However, it might be a good idea to create a post-commit hook to automatically format the project in the future.

@daniel-rossier

Copy link
Copy Markdown
Contributor

Please coud you create a branch with the add-on and then perform a PR from this branch? (this is actually our normal dev flow)

@daniel-rossierdaniel-rossier left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to follow our standard workflow: please create a branch associated to the issue

@AndreCostaaaAndreCostaaa mentioned this pull request Dec 16, 2024
2 tasks
@daniel-rossier

daniel-rossier commented Feb 5, 2025

Copy link
Copy Markdown
Contributor

This PR still needs rework.

@daniel-rossier

Copy link
Copy Markdown
Contributor

This is probably not valid any longer, right?

@AndreCostaaa

Copy link
Copy Markdown
Contributor

Closing this as #98 is now closed

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Karilla@daniel-rossier@AndreCostaaa