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

iox-#2327 Replace macro ENUM with IOX_ENUM and CLASS with IOX_CLASS #2328

iox-#2327 Remove macro ENUM and rename macro CLASS to IOX_C_CLASS

21f6d09
Select commit
Loading
Failed to load commit list.
Merged

iox-#2327 Replace macro ENUM with IOX_ENUM and CLASS with IOX_CLASS #2328

iox-#2327 Remove macro ENUM and rename macro CLASS to IOX_C_CLASS
21f6d09
Select commit
Loading
Failed to load commit list.
Task list completed / task-list-completed succeeded Aug 29, 2024 in 0s

19 / 19 tasks completed

All tasks have been completed

Details

Required Tasks

Task Status
Removing the ENUM macro, replacing its use with the enum keyword. Incomplete
Command: sed -i -e 's/\<CLASS\>/IOX_C_CLASS/g' $(find -type f) Incomplete
One file was reverted since CLASS was in a comment; not referencing the macro. Incomplete
Code follows the coding style of [CONTRIBUTING.md][contributing] Completed
Tests follow the [best practice for testing][testing] Completed
Changelog updated [in the unreleased section][changelog] including API breaking changes Completed
Branch follows the naming format (iox-123-this-is-a-branch) Completed
Commit messages have the issue ID (iox-#123 commit text) Completed
Commit author matches [Eclipse Contributor Agreement][eca] (and ECA is signed) Completed
Follow the same conventions as for commit messages Incomplete
Link to the relevant issue Incomplete
Relevant issues are linked Completed
Add sensible notes for the reviewer Completed
All checks have passed (except task-list-completed) Completed
Assign PR to reviewer Completed
Consider a second reviewer for complex new features or larger refactorings Completed
Commits are properly organized and messages are according to the guideline Completed
Code according to our coding style and naming conventions Completed
Unit tests have been written for new behavior Completed
Public API changes are documented via doxygen Completed
Copyright owner are updated in the changed files Completed
All touched (C/C++) source code files from iceoryx_hoofs have been added to ./clang-tidy-diff-scans.txt Completed
PR title describes the changes Completed
All open points are addressed and tracked via issues Completed
Closes #2327 Incomplete