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 #30, Update config files to match naming convention #32

Merged
merged 1 commit into from
May 31, 2023

Conversation

jphickey
Copy link
Collaborator

Describe the contribution
Updates BP config file structure to adhere to the naming/content conventions prescribed in the CFE documentation. This is primarily moving content between headers, not changing the content itself.

Fixes #30

Testing performed
Build and run BP tests

Expected behavior changes
None (same config, affects source file structure only)

System(s) tested on
Debian

Additional context
See e.g. HS app for reference

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

Updates BP config file structure to adhere to the naming/content
conventions prescribed in the CFE documentation.  This is primarily
moving content between headers, not changing the content itself.
@jphickey jphickey requested a review from glan6388 May 31, 2023 14:33
@jphickey jphickey added the ccb:approved Pull request has been reviewed and approved for merging label May 31, 2023
@dzbaker dzbaker merged commit 5b38642 into nasa:main May 31, 2023
@jphickey jphickey deleted the fix-30-configfiles branch September 21, 2023 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ccb:approved Pull request has been reviewed and approved for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update header file naming based on documented convention
3 participants