Skip to content

Remove retired Func-tastic references from help and docs - #4121

Open
dprotaso wants to merge 1 commit into
mainfrom
dprotaso/remove-func-tastic-references
Open

dprotaso wants to merge 1 commit into
mainfrom
dprotaso/remove-func-tastic-references

Conversation

@dprotaso

@dprotaso dprotaso commented Nov 1, 2026

Copy link
Copy Markdown
Member

Remove references to the retiring Func-tastic repository from CLI help and language-pack documentation, as tracked in knative/community#1802.

Use clearly illustrative repository/template names in CLI examples, retain generic template-management guidance, and regenerate the command reference. No runtime behavior changes.

Validation:

  • ./hack/update-codegen.sh
  • make check
  • make test (core tests with race detection)
  • Confirmed no Func-tastic references remain in CLI help or docs.

@knative-prow

knative-prow Bot commented Nov 1, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprotaso

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:

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

@knative-prow knative-prow Bot added approved 🤖 PR has been approved by an approver from all required OWNERS files. size/M 🤖 PR changes 30-99 lines, ignoring generated files. labels Nov 1, 2026
@dprotaso

dprotaso commented Nov 1, 2026

Copy link
Copy Markdown
Member Author

/hold

@knative-prow knative-prow Bot added the do-not-merge/hold 🤖 PR should not merge because someone has issued a /hold command. label Nov 1, 2026
@codecov

codecov Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.84%. Comparing base (6bbfab4) to head (aa67831).

❗ There is a different number of reports uploaded between BASE (6bbfab4) and HEAD (aa67831). Click for more details.

HEAD has 12 uploads less than BASE
Flag BASE (6bbfab4) HEAD (aa67831)
unit macos-14 1 0
integration 2 1
e2e 4 2
e2e-config-ci 1 0
e2e springboot 1 0
e2e node 1 0
e2e typescript 1 0
e2e rust 1 0
e2e go 1 0
e2e quarkus 1 0
e2e python 1 0
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4121      +/-   ##
==========================================
- Coverage   58.64%   52.84%   -5.80%     
==========================================
  Files         221      221              
  Lines       27633    27635       +2     
==========================================
- Hits        16204    14605    -1599     
- Misses      10054    11914    +1860     
+ Partials     1375     1116     -259     
Flag Coverage Δ
e2e 19.54% <100.00%> (-10.54%) ⬇️
e2e go ?
e2e node ?
e2e python ?
e2e quarkus ?
e2e rust ?
e2e springboot ?
e2e typescript ?
e2e-config-ci ?
integration 28.72% <ø> (+12.63%) ⬆️
unit macos-14 ?
unit macos-latest 49.91% <100.00%> (+<0.01%) ⬆️
unit ubuntu-24.04-arm 50.21% <100.00%> (-0.01%) ⬇️
unit ubuntu-latest 50.63% <100.00%> (+<0.01%) ⬆️
unit windows-latest 49.16% <100.00%> (+<0.01%) ⬆️

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

☔ View full report in Codecov by Harness.
📢 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved 🤖 PR has been approved by an approver from all required OWNERS files. do-not-merge/hold 🤖 PR should not merge because someone has issued a /hold command. size/M 🤖 PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant