feat: make templates consistent across runtimes

[lance]

Changes

• 🎁 Ensures that the templates for all runtimes are consistent with Provide a consistent developer experience for all function templates #572
  • Highlight where to put business logic
  • Effective, useful and descriptive comments
  • Template functions echo their input
  • All templates have unit tests

/kind enhancement

Fixes: #572

Release Note

Aligns all templates to a common standard behavior and comments

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lance

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [lance]

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

[codecov]

Codecov Report

Merging #948 (677314e) into main (739cded) will increase coverage by 1.98%.
The diff coverage is 29.13%.

@@            Coverage Diff             @@
##             main     #948      +/-   ##
==========================================
+ Coverage   43.85%   45.83%   +1.98%     
==========================================
  Files          55       57       +2     
  Lines        5163     7442    +2279     
==========================================
+ Hits         2264     3411    +1147     
- Misses       2576     3703    +1127     
- Partials      323      328       +5     

Impacted Files	Coverage Δ
cmd/completion_util.go	0.00% <0.00%> (ø)
cmd/invoke.go	34.81% <0.00%> (-0.97%)	⬇️
docker/docker_client.go	82.52% <ø> (+0.57%)	⬆️
k8s/logs.go	0.00% <0.00%> (ø)
k8s/secrets.go	13.54% <0.00%> (-0.75%)	⬇️
s2i/builder.go	32.69% <32.69%> (ø)
client.go	61.94% <50.00%> (+2.01%)	⬆️
invoke.go	52.99% <61.53%> (+2.99%)	⬆️
cmd/build.go	60.49% <70.83%> (+0.86%)	⬆️
cmd/root.go	72.28% <100.00%> (+1.66%)	⬆️
... and 55 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1ee9fa1...677314e. Read the comment docs.

[matejvasek]

/lgtm
/hold for tests

[zroubalik]

Looking good, just a small nit. I'd add blank lines around the comment, to make it more prominent and easily recognizable for users when the open the source code for the first time.

[matejvasek]

@lance there are style issues, and the embedded templates contains some target/* files that shouldn't be there.

[matejvasek]

There is trailing white space in templates/python/cloudevents/func.py:11:.

[lkingland]

/lgtm as well!

[matejvasek]

/unhold

[matejvasek]

/lgtm