Skip to content

Conversation

@ghost ghost self-requested a review as a code owner January 14, 2025 13:05
@ghost ghost mentioned this pull request Jan 14, 2025
name: "${{matrix.os}}, ${{matrix.build_type}}, shared=${{matrix.shared_library}}"
# run only on upstream; forks will not have the HW
if: github.repository == 'oneapi-src/unified-memory-framework'

Copy link
Contributor

Choose a reason for hiding this comment

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

pls update comment on top:

# Runs tests on multi-numa machine -> # Runs tests on multi-numa machines (plural)

name: "${{matrix.os}}, ${{matrix.build_type}}, shared=${{matrix.shared_library}}"
# run only on upstream; forks will not have the HW
if: github.repository == 'oneapi-src/unified-memory-framework'

Copy link
Contributor

Choose a reason for hiding this comment

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

as I mentioned before - pls update commits, to only have a single one, with proper description. Message Update reusable_multi_numa.yml doesn't say much 😉

@ghost ghost mentioned this pull request Jan 14, 2025
@ghost ghost closed this Jan 14, 2025
This pull request was closed.
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.

1 participant