Skip to content

Conversation

@lcarcaramo
Copy link
Member

💡 Issue Reference

Issue: N/A

💻 What does this address?

  • Make dev current with main. I don't know how it got out of sync.

📟 Implementation Details

N/A

📋 Is there a test case?

N/A

ElijahSwiftIBM and others added 26 commits January 28, 2025 16:38
-Added trailing newlines to changed files
-Removed extraneous trailing newlines from one file
Use const for parameters and members where appropriate
use static for member function where appropriate
-Fix const parms in logger
-Adjust makefile for running on platform
-Update makefile to work with z-open make and cppcheck
- mover variable definition to where it's only defined in a path where it is used
- Change rc checking logic to remove redundancy
Signed-off-by: Leonard Carcaramo <[email protected]>
Signed-off-by: Leonard Carcaramo <[email protected]>
Signed-off-by: Leonard Carcaramo <[email protected]>
Signed-off-by: Leonard Carcaramo <[email protected]>
@lcarcaramo lcarcaramo self-assigned this Feb 20, 2025
Copy link
Collaborator

@ElijahSwiftIBM ElijahSwiftIBM left a comment

Choose a reason for hiding this comment

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

Looks good

@lcarcaramo lcarcaramo merged commit 28bfcd6 into dev Feb 20, 2025
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