chore: refactor repository layout

[zroubalik]

Signed-off-by: Zbynek Roubalik zroubalik@gmail.com

I'd prefer if we can merge this PR asap, rebasing this will be very painful :))

Moved majority of code into pkg. This is the layout now:

$ tree -L 2 -d 
.
├── bin
├── cmd
│   ├── func
│   ├── templates
│   └── testdata
├── docs
│   ├── building-functions
│   ├── function-templates
│   ├── generator
│   ├── language-packs
│   └── reference
├── generate
│   └── templates
├── hack
├── pkg
│   ├── builders
│   ├── config
│   ├── docker
│   ├── filesystem
│   ├── function
│   ├── http
│   ├── k8s
│   ├── knative
│   ├── mock
│   ├── openshift
│   ├── pipelines
│   ├── progress
│   ├── ssh
│   ├── testing
│   └── utils
├── plugin
├── schema
│   └── generator
├── templates
│   ├── go
│   ├── node
│   ├── python
│   ├── quarkus
│   ├── rust
│   ├── springboot
│   └── typescript
├── test
│   ├── common
│   ├── e2e
│   └── oncluster
├── third_party
│   └── VENDOR-LICENSE
└── vendor

Changes

• 🧹 refactor repository layout

Fixes: #1547

[codecov]

Codecov Report

Base: 60.87% // Head: 60.64% // Decreases project coverage by -0.23% ⚠️

Coverage data is based on head (5e5a1a3) compared to base (e6618c0).
Patch coverage: 77.50% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1554      +/-   ##
==========================================
- Coverage   60.87%   60.64%   -0.23%     
==========================================
  Files          73       76       +3     
  Lines       10150    10216      +66     
==========================================
+ Hits         6179     6196      +17     
- Misses       3399     3453      +54     
+ Partials      572      567       -5     

Flag	Coverage Δ
integration-tests	50.93% <61.53%> (-0.45%)	⬇️
unit-tests	?
unit-tests-macos-latest	49.40% <77.50%> (?)
unit-tests-ubuntu-latest	50.90% <77.50%> (?)
unit-tests-windows-latest	49.27% <65.00%> (?)

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

Impacted Files	Coverage Δ
cmd/build.go	72.10% <ø> (ø)
cmd/client.go	83.95% <ø> (ø)
cmd/completion_util.go	0.00% <ø> (ø)
cmd/config.go	30.18% <ø> (ø)
cmd/config_envs.go	25.40% <ø> (ø)
cmd/config_labels.go	73.03% <ø> (ø)
cmd/config_volumes.go	25.13% <ø> (ø)
cmd/create.go	46.96% <ø> (ø)
cmd/delete.go	64.10% <ø> (ø)
cmd/deploy.go	68.45% <ø> (ø)
... and 69 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[lkingland]

I think the part I like about this the most is that, by default, we are back to there being a sane, Go-friendly default import path for users of the client library:

import knative.dev/func/pkg/function
...

function.XYZ

This is perhaps a bit pedantic, but since the singular makes it sound like they are expecting to operate on a single function, and the name is "Knative Functions", would it be better for the user if the package name were plural?

functions.NewFunction?
functions.XYZ?

Overall huge improvement. Yes' let's merge quickly to minimize collateral damage.

/lgtm
/hold for answer if anyone else thinks functions plural

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lkingland, zroubalik

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 [lkingland,zroubalik]

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