Skip to content

DO NOT MERGE: CI Cache testing#5612

Open
johnwparent wants to merge 2 commits into
spack:developfrom
johnwparent:engflow-cache-test
Open

DO NOT MERGE: CI Cache testing#5612
johnwparent wants to merge 2 commits into
spack:developfrom
johnwparent:engflow-cache-test

Conversation

@johnwparent

@johnwparent johnwparent commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@johnwparent
Init cmake-re testing
ccc40a4 
Signed-off-by: John Parent <john.parent@kitware.com>
@spackbot-triage spackbot-triage Bot added new-package Additions of new packages to the Spack ecosystem new-version Modifications to packages' `depends_on()` directives dependencies Modifications with a `depends_on()` directive patch Modifications to packages' sets of defined patches conflicts Modifications with a `conflicts()` directive maintainers Modifications to a package's `maintainers()` directive licenses Modifications to a package's `license()` directive build-systems Related to package build systems ci Related to CI/CD workflows and pipelines new-variant Modifications to packages' `variant()` directives stand-alone-tests Stand-alone (or smoke) tests for installed packages extends labels Jul 16, 2026
@johnwparent

johnwparent commented Jul 16, 2026

Copy link
Copy Markdown
Contributor Author

@spackbot fix style

@spackbot-app

spackbot-app Bot commented Jul 16, 2026

Copy link
Copy Markdown

Let me see if I can fix that for you!

@spackbot-app

spackbot-app Bot commented Jul 16, 2026

Copy link
Copy Markdown

I was able to run spack style --fix for you!

spack style --fix 
repos/spack_repo/builtin/build_systems/cmake_re.py
repos/spack_repo/builtin/packages/tipi_paraview/package.py
�[1;34m==> �[0mrunning ruff format
1 file reformatted, 1 file left unchanged
�[1;34m==> �[0mrunning ruff check
All checks passed!
�[1;34m==> �[0mstyle checks passed
Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request. If I was able to push to your branch, if you make further changes you will need to pull from your updated branch before pushing again.

I've updated the branch with style fixes.

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

Reviewers

@kwryankrattiger kwryankrattiger Awaiting requested review from kwryankrattiger kwryankrattiger is a code owner
@zackgalbreath zackgalbreath Awaiting requested review from zackgalbreath zackgalbreath is a code owner
@eugeneswalker eugeneswalker Awaiting requested review from eugeneswalker eugeneswalker is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

build-systems Related to package build systems ci Related to CI/CD workflows and pipelines conflicts Modifications with a `conflicts()` directive dependencies Modifications with a `depends_on()` directive extends licenses Modifications to a package's `license()` directive maintainers Modifications to a package's `maintainers()` directive new-package Additions of new packages to the Spack ecosystem new-variant Modifications to packages' `variant()` directives new-version Modifications to packages' `depends_on()` directives patch Modifications to packages' sets of defined patches stand-alone-tests Stand-alone (or smoke) tests for installed packages

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@johnwparent