Skip to content

py-non-regression-test-tools: add v1.1.4 - #47520

Merged
wdconinc merged 1 commit into
spack:developfrom
tech-91:package/py-non-regression-test-tools
Nov 12, 2024
Merged

py-non-regression-test-tools: add v1.1.4#47520
wdconinc merged 1 commit into
spack:developfrom
tech-91:package/py-non-regression-test-tools

Conversation

@tech-91

@tech-91 tech-91 commented Nov 9, 2024

Copy link
Copy Markdown
Contributor

Can you merge this PR?

@wdconinc wdconinc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. Please also add minimum version on py-setuptools dependency.

Comment thread var/spack/repos/builtin/packages/py-non-regression-test-tools/package.py Outdated
@wdconinc wdconinc self-assigned this Nov 10, 2024
@tech-91
tech-91 force-pushed the package/py-non-regression-test-tools branch from 8a20223 to 4b5b1c2 Compare November 11, 2024 04:09

@wdconinc wdconinc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. Technically, the py-setuptools limit for 1.1.2 is now too high, but there are sufficient py-setuptools version to satisfy that.

@wdconinc
wdconinc enabled auto-merge (squash) November 12, 2024 00:36
@wdconinc
wdconinc merged commit 31b2b79 into spack:develop Nov 12, 2024
@tech-91
tech-91 deleted the package/py-non-regression-test-tools branch November 12, 2024 07:51
@tech-91
tech-91 restored the package/py-non-regression-test-tools branch November 12, 2024 07:51
@tech-91
tech-91 deleted the package/py-non-regression-test-tools branch November 12, 2024 07:52
fryeguy52 pushed a commit to fryeguy52/spack that referenced this pull request Dec 17, 2024
Co-authored-by: t. chantrait <teddy.chantrait@cea.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants