astrolabsoftware / fink-broker

Astronomy Broker based on Apache Spark
https://fink-broker.org
Apache License 2.0
68 stars 13 forks source link

877 automated e2e tests #886

Closed fjammes closed 2 days ago

fjammes commented 5 days ago

IMPORTANT: Please create an issue first before opening a Pull Request. Linked to issue(s): Closes #877

What changes were proposed in this pull request?

Tools to run self-hosted CI on a linux server and GHA integration.

How was this patch tested?

On fink server @lpc

codecov[bot] commented 2 days ago

Codecov Report

Attention: Patch coverage is 74.64789% with 36 lines in your changes missing coverage. Please review.

Project coverage is 84%. Comparing base (70a2540) to head (0a7a209). Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
fink_broker/mm_utils.py 72% 16 Missing :warning:
bin/raw2science.py 59% 15 Missing :warning:
bin/distribute.py 80% 4 Missing :warning:
fink_broker/spark_utils.py 66% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #886 +/- ## ===================================== - Coverage 84% 84% -1% ===================================== Files 21 21 Lines 1292 1330 +38 ===================================== + Hits 1092 1121 +29 - Misses 200 209 +9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

sonarcloud[bot] commented 2 days ago

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

fjammes commented 2 days ago

@JulienPeloton , FYI I have added a "noscience" pipeline, because the Github one is down.

fjammes commented 2 days ago

LGTM

JulienPeloton commented 1 day ago

Why did you close this PR?

fjammes commented 1 day ago

My mistake, I wanted to merge it, I have merged it manually...

JulienPeloton commented 1 day ago

:D