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

Fix #37, Add style configuration #38

Merged
merged 1 commit into from
Jan 14, 2020
Merged

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Jan 6, 2020

Describe the contribution
Fixes #37

Testing performed
See as applied in nasa/ci_lab#30
Enforced in nasa/ci_lab#28

Expected behavior changes

  • No impact to software behavior
  • Allows for application and enforcement of standard style

System(s) tested on

Additional context
To use, requires installation of clang-format-10 (minimum version needed for define alignment)
See .travis.yml in ci_lab pull request nasa/ci_lab#28

Contributor Info
Jacob Hageman - NASA/GSFC

@skliper
Copy link
Contributor Author

skliper commented Jan 10, 2020

Updated style for 120 line width and comment reflow (fixes unaligned comments)

@skliper
Copy link
Contributor Author

skliper commented Jan 14, 2020

Rebased

@skliper
Copy link
Contributor Author

skliper commented Jan 14, 2020

Note this was CCB 20200108 - reviewed and approved for IC branch, but updated for comment demangling.

@skliper skliper changed the base branch from master to ic-20200108 January 14, 2020 21:44
@skliper skliper merged commit 09d4221 into ic-20200108 Jan 14, 2020
@astrogeco astrogeco deleted the fix37-add-style-cfg branch February 26, 2020 06:21
@astrogeco astrogeco restored the fix37-add-style-cfg branch February 26, 2020 06:21
@astrogeco astrogeco deleted the fix37-add-style-cfg branch March 10, 2020 18:01
@skliper skliper added this to the 6.8.0 milestone Jun 1, 2020
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.

None yet

1 participant