Skip to content

chore: update manifest.yamls#846

Merged
knative-prow-robot merged 2 commits intoknative:mainfrom
lance:lance/tweak-manifest-yamls
Feb 22, 2022
Merged

chore: update manifest.yamls#846
knative-prow-robot merged 2 commits intoknative:mainfrom
lance:lance/tweak-manifest-yamls

Conversation

@lance
Copy link
Contributor

@lance lance commented Feb 22, 2022

Changes

  • 🐛 In a couple of cases, the invoke parameter was missing.
  • 🐛 There was an outdated use of a typescript buildpack in the typescript/cloudevent templates.

Signed-off-by: Lance Ball lball@redhat.com

/kind bug

Release Note

Fixes missing `cloudevent` invocation format for Node.js, SpringBoot and Rust CloudEvent templates

@lance
chore: update manifest.yamls
e396875 
In a couple of cases, the `invoke` parameter was missing. And there was an
outdated use of a typescript buildpack in the typescript/cloudevent templates.

Signed-off-by: Lance Ball <lball@redhat.com>
@lance lance requested a review from matejvasek February 22, 2022 16:38
@knative-prow-robot knative-prow-robot added kind/bug Bugs size/S 🤖 PR changes 10-29 lines, ignoring generated files. labels Feb 22, 2022
@knative-prow-robot
Copy link

knative-prow-robot commented Feb 22, 2022

[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:

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

@knative-prow-robot knative-prow-robot added the approved 🤖 PR has been approved by an approver from all required OWNERS files. label Feb 22, 2022
@matejvasek
Copy link
Contributor

matejvasek commented Feb 22, 2022

@lance were are the TS changes?

@matejvasek matejvasek requested a review from lkingland February 22, 2022 16:45
@matejvasek
Copy link
Contributor

matejvasek commented Feb 22, 2022

/hold

@knative-prow-robot knative-prow-robot added the do-not-merge/hold 🤖 PR should not merge because someone has issued a /hold command. label Feb 22, 2022
@lance
fixup: I should set VSCode to autosave on focus change
462696f 
Signed-off-by: Lance Ball <lball@redhat.com>
@knative-prow-robot knative-prow-robot added size/M 🤖 PR changes 30-99 lines, ignoring generated files. and removed size/S 🤖 PR changes 10-29 lines, ignoring generated files. labels Feb 22, 2022
@lance
Copy link
Contributor Author

lance commented Feb 22, 2022

@matejvasek here: 462696f

@matejvasek
Copy link
Contributor

matejvasek commented Feb 22, 2022

fixup: I should set VSCode to autosave on focus change

I learned obsessively press CTRL+S everywhere 😄

@codecov
Copy link

codecov bot commented Feb 22, 2022

Codecov Report

Merging #846 (462696f) into main (5fad42b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #846   +/-   ##
=======================================
  Coverage   43.36%   43.36%           
=======================================
  Files          48       48           
  Lines        4690     4690           
=======================================
  Hits         2034     2034           
  Misses       2367     2367           
  Partials      289      289

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 5fad42b...462696f. Read the comment docs.

@matejvasek
Copy link
Contributor

matejvasek commented Feb 22, 2022

/unhold

@knative-prow-robot knative-prow-robot removed the do-not-merge/hold 🤖 PR should not merge because someone has issued a /hold command. label Feb 22, 2022
@matejvasek
Copy link
Contributor

matejvasek commented Feb 22, 2022

/lgtm

@knative-prow-robot knative-prow-robot added the lgtm 🤖 PR is ready to be merged. label Feb 22, 2022
@knative-prow-robot knative-prow-robot merged commit ad42f3a into knative:main Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@matejvasek matejvasek Awaiting requested review from matejvasek

@lkingland lkingland Awaiting requested review from lkingland

Assignees

@matejvasek matejvasek

Labels

approved 🤖 PR has been approved by an approver from all required OWNERS files. kind/bug Bugs lgtm 🤖 PR is ready to be merged. size/M 🤖 PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lance @knative-prow-robot @matejvasek