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.

Sign up for GitHub

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

Jump to bottom

Fix #37, Apply style #92

Merged
merged 1 commit into from
Aug 28, 2020
Merged

Fix #37, Apply style #92

merged 1 commit into from
Aug 28, 2020

Conversation

astrogeco
Copy link
Contributor

@astrogeco astrogeco commented Aug 27, 2020 •
edited
Loading

Describe the contribution
Fix #37 - applies standard coding style

Ran

find . -name "*.[ch]" -exec clang-format -i --style=file  {} +

Testing performed
CI only, whitespace.

Expected behavior changes
None

System(s) tested on
CI

Additional context
Redoing #88 which was accidentally merged and fixing to avoid conflicts

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman, NASA-GSFC
Gerardo E. Cruz-Ortiz, NASA-GSFC

@astrogeco astrogeco changed the base branch from main to integration-candidate August 27, 2020 23:02
@astrogeco astrogeco mentioned this pull request Aug 27, 2020
Closed
@yammajamma yammajamma added the CCB:Approved Indicates code approval by CCB label Aug 28, 2020
@yammajamma yammajamma merged commit f0c26b6 into integration-candidate Aug 28, 2020
@yammajamma yammajamma mentioned this pull request Aug 28, 2020
Merged
@skliper skliper deleted the apply-style-again branch February 1, 2021 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yammajamma yammajamma yammajamma approved these changes

Assignees
No one assigned
Labels
CCB:Approved Indicates code approval by CCB
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Apply code style
2 participants
@astrogeco @yammajamma