-
-
Notifications
You must be signed in to change notification settings - Fork 7
Compile valgrind wrapper #5
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.
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
Conversation
|
@conda-forge-admin, please rerender |
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
…onda-forge-pinning 2024.12.15.16.05.58
|
This is ready now @xhochy @kenodegard, thanks! Oops wait, this was |
|
@conda-forge-admin, please rerender |
|
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipe/meta.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12350918146. Examine the logs at this URL for more detail. |
…onda-forge-pinning 2024.12.15.16.05.58
|
@conda-forge-admin, please rerender |
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
…onda-forge-pinning 2024.12.15.16.05.58
|
Is this only visible with an import test? Couldn't we also execute something what did break in your job directly? |
|
We'd need to create a pytest test file and run it. Not against it, but the import test looked simpler. |
|
@conda-forge-admin, please rerender |
|
We probably need something here like we do for |
…onda-forge-pinning 2024.12.15.16.05.58
|
Upstream only publishes linux wheels now, though. I'm not even sure this works outside Linux environments. |
xhochy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's merge and see whether someone requests osx support.
|
Hey just discovered this repo. As you mentionned, we just switched to prebuilt wheels in pytest-codspeed [v3.1.0]9https://github.com/CodSpeedHQ/pytest-codspeed/releases/tag/v3.1.0).
It works outside linux environment however the instrumentation capabilities are not enabled. |
Checklist
0(if the version changed)conda-smithy(Use the phrase@conda-forge-admin, please rerenderin a comment in this PR for automated rerendering)Relevant refs upstream: