Skip to content

Commit

Permalink
Semgrep is not available on Windows
Browse files Browse the repository at this point in the history
  • Loading branch information
nicolargo committed Dec 29, 2024
1 parent 797b0d0 commit b77d1b3
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ jobs:
# Error: Process completed with exit code 1.
test-windows:

needs: source-code-checks
# https://github.com/actions/runner-images?tab=readme-ov-file#available-images
runs-on: windows-2022
strategy:
Expand Down
2 changes: 1 addition & 1 deletion dev-requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,6 @@ pytest
requirements-parser
ruff
selenium
semgrep
semgrep; platform_system == 'Linux'
setuptools>=65.5.1 # not directly required, pinned by Snyk to avoid a vulnerability
webdriver-manager
1 change: 1 addition & 0 deletions tests/test_memoryleak.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,4 +32,5 @@ def test_memoryleak(glances_stats_no_history, logger):
snapshot_end = tracemalloc.take_snapshot()
snapshot_diff = snapshot_end.compare_to(snapshot_begin, 'filename')
memory_leak = sum([s.size_diff for s in snapshot_diff]) // iteration
logger.info('Memory consume per iteration: {memory_leak} bytes')
assert memory_leak < 1000, f'Memory leak: {memory_leak} bytes'

0 comments on commit b77d1b3

Please sign in to comment.