Skip to content

Conversation

LittleHuba
Copy link
Contributor

No description provided.

@LittleHuba LittleHuba requested a review from a team as a code owner August 11, 2025 15:37
@LittleHuba LittleHuba linked an issue Aug 11, 2025 that may be closed by this pull request
2 tasks
Copy link

The created documentation from the pull request is available at: docu-html

Copy link
Contributor

@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

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

I am fine from process and safety point of view, Maybe add a SW development expert to the review?

Further, handle errors internally if feasible but bubble an error up if a user should know about it.

Throwing exceptions should be avoided, as they normally come with heap allocation.
In C++ exceptions are therefore hardwired to an abort.
Copy link
Contributor

Choose a reason for hiding this comment

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

I thought only in the project specific configuration of C++ (lib)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved to C++. And yes only for this project, but this guideline is only for this project.

@LittleHuba LittleHuba force-pushed the ulhu_api_guidelines branch from 62f18e5 to 7238842 Compare August 14, 2025 14:54
* initialization of variables.

For this project an code style is not yet decided.
For this project an code style is not yet fully decided.
Copy link

Choose a reason for hiding this comment

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

a code style / a coding style

Copy link

This PR is stale because it has been open for 30 days with no activity. It will be closed in 10 days if no further activity occurs. #magic___^_^___line

Copy link

github-actions bot commented Oct 1, 2025

⚠️ Docs-as-Code version mismatch detected
Please check the CI build logs for details and align the documentation version with the Bazel dependency.

@LittleHuba LittleHuba removed the Stale label Oct 1, 2025
@LittleHuba LittleHuba force-pushed the ulhu_api_guidelines branch from 6156bcc to d7d0ca2 Compare October 1, 2025 13:57
@LittleHuba LittleHuba force-pushed the ulhu_api_guidelines branch from d7d0ca2 to c42bb9f Compare October 1, 2025 14:28
Sign up for free to 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.

Improvement: Add an API implementation guideline
4 participants