Skip to content
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

start using proxyspy #131

Merged
merged 2 commits into from
Feb 8, 2025
Merged

start using proxyspy #131

merged 2 commits into from
Feb 8, 2025

Conversation

mcg1969
Copy link
Collaborator

@mcg1969 mcg1969 commented Feb 7, 2025

I've also removed the tests from the conda recipe since we're running them so comprehensively here in CI.

@mcg1969 mcg1969 force-pushed the proxyspy branch 2 times, most recently from a875601 to dfd72d8 Compare February 7, 2025 22:19
@@ -31,7 +31,7 @@ jobs:
run: |
source $CONDA/etc/profile.d/conda.sh
conda install conda-build
conda build conda.recipe --no-test
conda build conda.recipe
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

we've reduce the tests in the conda recipe to basic imports so we might as well leave them on

@jezdez jezdez merged commit 933ba74 into main Feb 8, 2025
48 checks passed
@jezdez jezdez deleted the proxyspy branch February 8, 2025 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants