search

knative / docs

User documentation for Knative components.
https://knative.dev/docs/
Other
4.55k stars 1.23k forks source link

doc: Mark TriggerMesh as Defunct #6154

Open zacheryph opened 3 weeks ago

zacheryph commented 3 weeks ago

Proposed Changes

Sadly Triggermesh shuttered a year ago, last fall. It is sad to see them gone, though these probably should not be marked as active anymore.

I wonder if these should simply be removed or possibly add a 3rd section for.. Historic? Defunct? not sure what to name it 🤔

Reference: https://github.com/triggermesh/triggermesh/issues/1544#issuecomment-2314595965

knative-prow[bot] commented 3 weeks ago

Welcome @zacheryph! It looks like this is your first PR to knative/docs 🎉

knative-prow[bot] commented 3 weeks ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: zacheryph Once this PR has been reviewed and has the lgtm label, please assign nainaz for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/knative/docs/blob/main/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
netlify[bot] commented 3 weeks ago

Deploy Preview for knative ready!

Built without sensitive environment variables

Name Link
Latest commit 4a04d9e88467814c5aa82cbf6e35cfe5d1728bb9
Latest deploy log https://app.netlify.com/sites/knative/deploys/673c9cdac913850008850323
Deploy Preview https://deploy-preview-6154--knative.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

pierDipi commented 2 weeks ago

I wonder if these should simply be removed or possibly add a 3rd section for.. Historic? Defunct? not sure what to name it

Thanks @zacheryph, I would remove them.

In the meantime, we're working on replacements and we would love feedback on which one you would be using here: https://github.com/knative/eventing/issues/8185

zacheryph commented 3 days ago

@pierDipi apologize, I missed notification for this one. Updated to remove Triggermesh. I will have to take a look at that issue.