search

openSUSE / openSUSE-release-tools

Tools to aid in staging and release work for openSUSE/SUSE
GNU General Public License v2.0
59 stars 92 forks source link

check_source.py: Don't skip various checks and reviews for non-spec packages #3017

Closed Vogtinator closed 10 months ago

Vogtinator commented 10 months ago

There was an early exit for packages which don't have a .spec file, but that is too early and skipped important parts like running the source_validator and download_files services and adding the review-team review.

codecov-commenter commented 10 months ago

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.01% :warning:

Comparison is base (fe4936c) 28.48% compared to head (2a369ec) 28.48%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3017 +/- ## ========================================== - Coverage 28.48% 28.48% -0.01% ========================================== Files 85 85 Lines 14717 14715 -2 ========================================== - Hits 4192 4191 -1 + Misses 10525 10524 -1 ``` | [Files Changed](https://app.codecov.io/gh/openSUSE/openSUSE-release-tools/pull/3017?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openSUSE) | Coverage Δ | | |---|---|---| | [check\_source.py](https://app.codecov.io/gh/openSUSE/openSUSE-release-tools/pull/3017?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openSUSE#diff-Y2hlY2tfc291cmNlLnB5) | `70.18% <100.00%> (+0.07%)` | :arrow_up: |

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