Skip to content

Pub fixes#11538

Open
sschuberth wants to merge 9 commits intomainfrom
pub-fixes
Open

Pub fixes#11538
sschuberth wants to merge 9 commits intomainfrom
pub-fixes

Conversation

@sschuberth
Copy link
Member

Please have a look at the individual commit messages for the details.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
This allows for an upcoming fix.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
Make properties that are in any case set `lateinit` and reorder them for
readability.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
@sschuberth sschuberth requested a review from a team as a code owner March 5, 2026 17:20
@sschuberth
Copy link
Member Author

FYI @maennchen, see the last commit.

@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.98%. Comparing base (6159101) to head (8a48477).

Additional details and impacted files
@@            Coverage Diff            @@
##               main   #11538   +/-   ##
=========================================
  Coverage     57.97%   57.98%           
  Complexity     1730     1730           
=========================================
  Files           349      349           
  Lines         12993    12992    -1     
  Branches       1263     1262    -1     
=========================================
  Hits           7533     7533           
  Misses         4995     4995           
+ Partials        465      464    -1     
Flag Coverage Δ
funTest-external-tools 14.18% <ø> (+<0.01%) ⬆️
funTest-no-external-tools 30.87% <ø> (+<0.01%) ⬆️
test-ubuntu-24.04 42.55% <ø> (+<0.01%) ⬆️
test-windows-2025 42.53% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
This prematurely adds support for the upcoming PURL type for generic Git
repositories [1] to avoid an empty PURL being created from the `id` if
the VCS host cannot be determined.

[1]: package-url/purl-spec#823

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
This reverts commit 4af9208. It is not needed anymore as Pub SDK packages
are being filtered out before PURL construction, and Carthage supports
non-empty PURLs for generic Git URLs now.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants