-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for v1beta1 API #245
Conversation
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
…to v1beta1-api Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
Codecov Report
@@ Coverage Diff @@
## master #245 +/- ##
==========================================
- Coverage 71.15% 71.03% -0.12%
==========================================
Files 37 37
Lines 2368 2372 +4
Branches 397 400 +3
==========================================
Hits 1685 1685
- Misses 683 687 +4
Continue to review full report at Codecov.
|
@@ -3,6 +3,6 @@ kind: PipelineResource | |||
metadata: | |||
name: ${1:git-source} | |||
spec: | |||
type: ${2|git,image,pullRequest,cluster,storage|} | |||
type: ${2|git,image,pullRequest,cluster,storage,gcs,build-gcs,cloudevent|} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apiVersion should be tekton.dev/v1beta1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR upgrades snippet content, fix some indentation issues.
Snippets migrated to
v1beta1
version according to https://github.com/tektoncd/pipeline/blob/master/docs/migrating-v1alpha1-to-v1beta1.mdThis PR brings a new npm command to regenerate snippets json file.
Fix: #183
Also this PR fix: #184, #185, #225, #176