Skip to content

Conversation

@JaeseungYeom
Copy link
Contributor

margo and ucx headers were included even when neither was used.
I added a MACRO guard.

@JaeseungYeom JaeseungYeom requested a review from wangvsa June 17, 2025 22:38
@JaeseungYeom JaeseungYeom self-assigned this Jun 17, 2025
@JaeseungYeom JaeseungYeom added the bug Something isn't working label Jun 17, 2025
@JaeseungYeom
Copy link
Contributor Author

The CI error is fixed in a separate PR #142

Copy link
Collaborator

@wangvsa wangvsa 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 to me.

@wangvsa wangvsa merged commit 4632996 into flux-framework:main Jun 18, 2025
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants