Skip to content

Conversation

@cowsed
Copy link
Contributor

@cowsed cowsed commented Jan 8, 2026

Description

Frame of airbrake application to get started with

Type of change

New program setup

How Has This Been Tested?

Compile and run on native sim and control freak

Checklist:

  • New functionality is documented in the necessary spots (i.e new functions documented in the header)
  • Unit tests cover any new functionality or edge cases that the PR was meant to resolve (if applicable)
  • The CI checks are passing
  • I reviewed my own code in the GitHub diff and am sure that each change is intentional
  • I feel comfortable about this code flying in a rocket

@cowsed cowsed marked this pull request as ready for review January 8, 2026 22:25
@cowsed cowsed requested a review from a team as a code owner January 8, 2026 22:25
Copy link
Member

@AarC10 AarC10 left a comment

Choose a reason for hiding this comment

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

Image

Fix the styling issues and Ill approve

  • Missing N prefixes in namespaces and n_ in files
  • C++ functions should be camelcase. First letter in publicly accessible functions should also be capitalized

@cowsed
Copy link
Contributor Author

cowsed commented Jan 12, 2026

even tho the *_init are in a .cpp file, theyre extern "C" so i think it makes sense to keep them c case

@cowsed cowsed merged commit a00897f into main Jan 12, 2026
6 checks passed
@cowsed cowsed deleted the cowsed/airbrake_setup branch January 12, 2026 12:35
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.

3 participants