🧹 Removing the vendor dir 🎆

[cardil]

Changes

• 🧹 Removing the vendor dir 🎆

/kind cleanup

Tests knative/hack#311
Tests knative/hack#222
Related to knative-extensions/kn-plugin-event#307

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (26614eb) 60.64% compared to head (5963269) 62.75%.
Report is 13 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1966      +/-   ##
==========================================
+ Coverage   60.64%   62.75%   +2.11%     
==========================================
  Files         106      106              
  Lines       13693    13693              
==========================================
+ Hits         8304     8593     +289     
+ Misses       4569     4231     -338     
- Partials      820      869      +49     

Flag	Coverage Δ
e2e-test	36.46% <ø> (+1.27%)	⬆️
e2e-test-oncluster	30.50% <ø> (?)
e2e-test-oncluster-runtime	26.96% <ø> (?)
e2e-test-runtime-go	25.59% <ø> (?)
e2e-test-runtime-node	26.60% <ø> (?)
e2e-test-runtime-python	26.60% <ø> (?)
e2e-test-runtime-quarkus	26.71% <ø> (?)
e2e-test-runtime-rust	25.59% <ø> (?)
e2e-test-runtime-springboot	25.73% <ø> (?)
integration-tests	51.42% <ø> (+2.09%)	⬆️
unit-tests-macos-latest	48.21% <ø> (+0.02%)	⬆️
unit-tests-ubuntu-latest	48.98% <ø> (-0.03%)	⬇️
unit-tests-windows-latest	48.25% <ø> (ø)

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

see 15 files with indirect coverage changes

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

[cardil]

Error from https://github.com/knative/func/actions/runs/6108788480/job/16578494890?pr=1966#step:10:199 looks like infra related:

Error: preparing container for next step: creating build container: creating container: creating read-write layer with ID "75ac42ff146caa497d0a1b90a87bd6db05861249d055ee23e0557339b2161ec0": no space left on device

[cardil]

https://github.com/knative/func/actions/runs/6108788480/job/16580443298#step:10:1 once again: no space left on the device

[cardil]

This PR is looking good (apart from obvious infra failures - seen above).

However, I'll put this on hold, since I'll be on PTO next week.

/hold

[lkingland]

/lgtm

Thanks for the improvement! Re-running the E2E test...

[matejvasek]

@cardil to fix the failing test please rebase on main.

[knative-prow]

New changes are detected. LGTM label has been removed.

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: cardil

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment