Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added .clang-format file and associate github workflow action to auto… #18

Merged

Conversation

dheadrick1618
Copy link
Member

…mate testing of c/c++ code format and styling.

…mate testing of c/c++ code format and styling.
@dheadrick1618 dheadrick1618 linked an issue Jul 11, 2023 that may be closed by this pull request
@dheadrick1618 dheadrick1618 self-assigned this Jul 11, 2023
@dheadrick1618 dheadrick1618 requested review from rcunrau, a team and abhisheknaik96 and removed request for a team July 11, 2023 14:41
Copy link

@StevenKnudsen StevenKnudsen left a comment

Choose a reason for hiding this comment

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

Seems like work in progress, so no comments other than the burnside-specific text is likely a side-effect of testing, right?

@dheadrick1618
Copy link
Member Author

Seems like work in progress, so no comments other than the burnside-specific text is likely a side-effect of testing, right?

Yea it didn't seem right to apply changes to the burnwire component code given the scope of this specific issue. But I will format its contents in the next fprime component based issue.

@dheadrick1618 dheadrick1618 merged commit 6dfad6b into main Jul 11, 2023
@dheadrick1618 dheadrick1618 deleted the devin/testing/create_clang_auto_format_workspace_action branch July 11, 2023 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup C++ code formatting github workspace action automation.
2 participants