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.

Sign up for GitHub

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

Fix tool exit order #2662

Merged
merged 4 commits into from
Dec 4, 2024
+117 −25

Less technical changelog

b345fc0
Select commit
Loading
Failed to load commit list.
Refresh
Merged

Fix tool exit order #2662

Less technical changelog
b345fc0
Select commit
Loading
Failed to load commit list.
CTAO-DPPS-SonarQube / ctapipe Sonarqube Results failed Dec 3, 2024 in 15s

Quality Gate failed

Failed

  • 0.00% Security Hotspots Reviewed on New Code (is less than 100.00%)

Analysis Details

1 Issue

  • Bug 0 Bugs
  • Vulnerability 1 Vulnerability
  • Code Smell 0 Code Smells

Coverage and Duplications

  • Coverage 95.70% Coverage (94.00% Estimated after merge)
  • Duplications 0.00% Duplicated Code (0.70% Estimated after merge)

Project ID: cta-observatory_ctapipe_AY52EYhuvuGcMFidNyUs

View in SonarQube

Annotations

Check failure on line 564 in src/ctapipe/core/tests/test_tool.py

See this annotation in the file changed.

@ctao-dpps-sonarqube ctao-dpps-sonarqube / ctapipe Sonarqube Results

src/ctapipe/core/tests/test_tool.py#L564 

Make sure that sending signals is safe here.
View more details on CTAO-DPPS-SonarQube