Skip to content

Commit

Permalink
Merge branch 'main' into pre-commit-ci-update-config
Browse files Browse the repository at this point in the history
  • Loading branch information
marcoesters authored Jan 7, 2025
2 parents 87f10f5 + 1b3c801 commit 644a702
Show file tree
Hide file tree
Showing 4 changed files with 42 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ jobs:
- uses: conda-incubator/setup-miniconda@d2e6a045a86077fb6cad6f5adf368e9076ddaa8d # v3.1.0
with:
auto-activate-base: true
activate-environment: ""
activate-environment: "test"
run-post: false

- name: Install dependencies
Expand Down
19 changes: 19 additions & 0 deletions news/118-skip-uninstallation-caches-test
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
### Enhancements

* <news item>

### Bug fixes

* Skip `test_uninstallation_remove_caches` if not on CI or if `CONDA_PKGS_DIRS` is set. (#118)

### Deprecations

* <news item>

### Docs

* <news item>

### Other

* <news item>
19 changes: 19 additions & 0 deletions news/125-build-environment
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
### Enhancements

* <news item>

### Bug fixes

* <news item>

### Deprecations

* <news item>

### Docs

* <news item>

### Other

* Run CI builds in a separate conda environment. (#125)
3 changes: 3 additions & 0 deletions tests/test_uninstall.py
Original file line number Diff line number Diff line change
Expand Up @@ -297,11 +297,14 @@ def _shortcuts_found(shortcut_env: Path) -> list:
(True, False),
ids=("shared pkgs", "remove pkgs"),
)
@pytest.mark.skipif(not ON_CI, reason="CI only - may remove shared caches")
def test_uninstallation_remove_caches(
mock_system_paths: dict[str, Path],
tmp_env: TmpEnvFixture,
shared_pkgs: bool,
):
if "CONDA_PKGS_DIRS" in os.environ:
pytest.skip("Test will fail with CONDA_PKGS_DIRS set.")
# Set up notices
if ON_WIN:
try:
Expand Down

0 comments on commit 644a702

Please sign in to comment.