Skip to content

PERF: Avoid CI on RemoteModule hash changes #5753

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.

Sign up for GitHub

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

Jump to bottom
Merged
hjmjohnson merged 1 commit into from
Jan 30, 2026
Merged

PERF: Avoid CI on RemoteModule hash changes #5753

hjmjohnson merged 1 commit into from
Jan 30, 2026
+17 −0

Conversation

@hjmjohnson
Copy link
Member

@hjmjohnson hjmjohnson commented Jan 30, 2026

  • BUG: Update remotes with bug fixes related to cmake modernization
  • ENH: Require python3.10 or later for python wrapping.
  • ENH: Update CI and scripts to require Python 3.10,
  • PERF: Update CI paths to ignore remote module files

Depends on:

PR Checklist

@github-actions github-actions bot added type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Performance Improvement in terms of compilation or execution time area:Python wrapping Python bindings for a class type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct area:Remotes Issues affecting the Remote module labels Jan 30, 2026
@hjmjohnson
PERF: Update CI paths to ignore remote module files
f01386b 
The RemoteModules are not built as part of the CI
infrastucture, so changes to those files do not
benefit from running the various CI components.
@hjmjohnson hjmjohnson force-pushed the remote-module-hash-no-rebuild branch from 492b562 to f01386b Compare January 30, 2026 18:02
@hjmjohnson hjmjohnson self-assigned this Jan 30, 2026
@hjmjohnson hjmjohnson marked this pull request as ready for review January 30, 2026 18:02
@github-actions github-actions bot removed area:Python wrapping Python bindings for a class area:Remotes Issues affecting the Remote module labels Jan 30, 2026
dzenanz
dzenanz approved these changes Jan 30, 2026
View reviewed changes
Copy link
Member

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

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

It looks like remote modules are not tested in the CI.

@dzenanz dzenanz requested a review from thewtex January 30, 2026 18:54
@hjmjohnson hjmjohnson merged commit e7b45a5 into main Jan 30, 2026
17 checks passed
@hjmjohnson hjmjohnson deleted the remote-module-hash-no-rebuild branch January 30, 2026 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dzenanz dzenanz dzenanz approved these changes

@thewtex thewtex Awaiting requested review from thewtex

Assignees

@hjmjohnson hjmjohnson

Labels

type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Performance Improvement in terms of compilation or execution time type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hjmjohnson @dzenanz