Skip to content

Conversation

@KyleFromNVIDIA
Copy link
Member

No description provided.

@KyleFromNVIDIA KyleFromNVIDIA requested a review from a team as a code owner November 20, 2025 22:05
@KyleFromNVIDIA KyleFromNVIDIA added the improvement Improves an existing functionality label Nov 20, 2025
@KyleFromNVIDIA KyleFromNVIDIA requested review from AyodeAwe and removed request for a team November 20, 2025 22:05
@KyleFromNVIDIA KyleFromNVIDIA added non-breaking Introduces a non-breaking change DO NOT MERGE labels Nov 20, 2025
@KyleFromNVIDIA
Copy link
Member Author

Output from changed-files workflow with a modified test_cpp check: https://github.com/rapidsai/cudf/actions/runs/19553810638/job/55991598296?pr=20689#step:2:7

The above proves that this workflow works.

@KyleFromNVIDIA KyleFromNVIDIA marked this pull request as draft November 21, 2025 15:15
@KyleFromNVIDIA
Copy link
Member Author

@KyleFromNVIDIA KyleFromNVIDIA marked this pull request as ready for review November 21, 2025 15:45
@KyleFromNVIDIA
Copy link
Member Author

- name: Calculate changed files
id: changed-files
uses: step-security/changed-files@95b56dadb92a30ca9036f16423fd3c088a71ee94 # v46.0.5
uses: rapidsai/shared-actions/changed-files@changed-files
Copy link
Member Author

Choose a reason for hiding this comment

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

Make this change before merging:

Suggested change
uses: rapidsai/shared-actions/changed-files@changed-files
uses: rapidsai/shared-actions/changed-files@main

- name: Check changed files
id: changed-files
if: ${{ inputs.enable_check_version_files_changed }}
uses: rapidsai/shared-actions/changed-files@changed-files
Copy link
Member Author

Choose a reason for hiding this comment

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

Make this change before merging:

Suggested change
uses: rapidsai/shared-actions/changed-files@changed-files
uses: rapidsai/shared-actions/changed-files@main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO NOT MERGE improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants