Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix codeql github action #2285

Merged
merged 1 commit into from
Mar 27, 2024
Merged

Fix codeql github action #2285

merged 1 commit into from
Mar 27, 2024

Conversation

piyush-garg
Copy link
Contributor

This will fix codeql action as it was not able to scan with autobuild step failing

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Includes tests (if functionality changed/added)
  • Run the code checkers with make check
  • Regenerate the manpages, docs and go formatting with make generated
  • Commit messages follow commit message best practices

See the contribution guide
for more details.

Release Notes

Fix codeql github action

@tekton-robot tekton-robot added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label Mar 26, 2024
@tekton-robot tekton-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 26, 2024
@piyush-garg
Copy link
Contributor Author

/test pull-tekton-cli-integration-tests

1 similar comment
@piyush-garg
Copy link
Contributor Author

/test pull-tekton-cli-integration-tests

@tekton-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vdemeester

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

The pull request process is described here

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

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 26, 2024
@tekton-robot tekton-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 27, 2024
@piyush-garg
Fix codeql github action
39d8d1a 
This will fix codeql action as it was not able to scan
with autobuild step failing and also add cache
@pratap0007
Copy link
Contributor

/lgtm

@tekton-robot
Copy link
Contributor

@pratap0007: changing LGTM is restricted to collaborators

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@PuneetPunamiya
Copy link
Member

/lgtm

@tekton-robot
Copy link
Contributor

@PuneetPunamiya: changing LGTM is restricted to collaborators

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@piyush-garg
Copy link
Contributor Author

@vdemeester can you please check now, I have added the caching step

@vdemeester
Copy link
Member

/lgtm
/meow

@tekton-robot
Copy link
Contributor

@vdemeester: cat image

In response to this:

/lgtm
/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Mar 27, 2024
@tekton-robot tekton-robot merged commit fc036dd into tektoncd:main Mar 27, 2024
7 of 8 checks passed
piyush-garg added a commit that referenced this pull request May 13, 2024
@piyush-garg
New version v0.37.0
f8b6dc0 
#2281 | [Piyush Garg] Add 0.35.2 in LTS doc | 2024/03/26-00:24
#2282 | [Piyush Garg] Enable dependabot on github actions | 2024/03/26-09:04
#2284 | [dependabot[bot]] Bump actions/checkout from 2 to 4 | 2024/03/26-09:32
#2283 | [dependabot[bot]] Bump github/codeql-action from 2 to 3 | 2024/03/26-10:42
#2285 | [Piyush Garg] Fix codeql github action | 2024/03/27-15:42
#2286 | [dependabot[bot]] Bump github.com/golangci/golangci-lint from 1.57.1 to 1.57.2 in /tools | 2024/03/31-20:40
#2288 | [dependabot[bot]] Bump golang.org/x/term from 0.18.0 to 0.19.0 | 2024/04/06-04:01
#2289 | [dependabot[bot]] Bump github.com/docker/docker | 2024/04/15-12:57
#2290 | [dependabot[bot]] Bump github.com/docker/cli | 2024/04/15-14:39
#2291 | [Piyush Garg] Bump sigstore to v1.8.3 | 2024/04/16-04:49
null | [dependabot[bot]] Bump github.com/sigstore/cosign/v2 from 2.2.2 to 2.2.4 | 2024/04/16-12:41
null | [dependabot[bot]] Bump github.com/docker/docker | 2024/04/19-19:19
null | [dependabot[bot]] Bump github.com/docker/cli | 2024/04/19-20:01
null | [dependabot[bot]] Bump golang.org/x/net from 0.22.0 to 0.23.0 | 2024/04/22-07:23
null | [dependabot[bot]] Bump github.com/docker/cli | 2024/04/25-05:15
null | [dependabot[bot]] Bump github.com/docker/docker | 2024/04/25-09:47
null | [dependabot[bot]] Bump github.com/tektoncd/pipeline from 0.58.0 to 0.59.0 | 2024/04/29-03:47
null | [dependabot[bot]] Bump github.com/tektoncd/triggers from 0.26.1 to 0.26.2 | 2024/04/30-19:55
null | [dependabot[bot]] Bump github.com/docker/cli | 2024/05/01-21:15
null | [dependabot[bot]] Bump github.com/docker/docker | 2024/05/01-21:53
null | [Chmouel Boudjnah] Add -E flag to exit with the pr sate on unix shell | 2024/05/06-16:30
null | [dependabot[bot]] Bump golang.org/x/term from 0.19.0 to 0.20.0 | 2024/05/07-04:38
null | [dependabot[bot]] Bump github.com/golangci/golangci-lint from 1.57.2 to 1.58.0 in /tools | 2024/05/07-07:08
null | [Piyush Garg] Bump k8s.io deps to match with tektoncd/pipeline | 2024/05/08-02:00
null | [Piyush Garg] Bump golangci-lint to v0.58.0 | 2024/05/08-06:34
null | [Piyush Garg] Group docker dep updates | 2024/05/08-08:16
null | [dependabot[bot]] Bump k8s.io/cli-runtime from 0.28.5 to 0.28.9 | 2024/05/08-09:08
null | [Piyush Garg] Bump k8s.io/apimachinery to v0.28.9 | 2024/05/08-11:16
null | [Piyush Garg] Group all k8s dep together | 2024/05/08-11:16
null | [Piyush Garg] Bump go version to 1.22 | 2024/05/08-17:36
null | [Piyush Garg] Bump goreleaser for latest v1.25.1 | 2024/05/08-17:36
null | [dependabot[bot]] Bump github.com/tektoncd/hub from 1.16.0 to 1.17.0 | 2024/05/08-20:12
null | [dependabot[bot]] Bump github.com/golangci/golangci-lint from 1.58.0 to 1.58.1 in /tools | 2024/05/09-05:58
null | [dependabot[bot]] Bump the go-docker-dependencies group with 2 updates | 2024/05/09-12:38
null | [dependabot[bot]] Bump github.com/tektoncd/triggers from 0.26.2 to 0.27.0 | 2024/05/10-19:00

Signed-off-by: Piyush Garg <pgarg@redhat.com>
@BrewTestBot BrewTestBot mentioned this pull request May 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vdemeester vdemeester vdemeester approved these changes

@vinamra28 vinamra28 Awaiting requested review from vinamra28

Assignees

@vdemeester vdemeester

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@piyush-garg @tekton-robot @pratap0007 @PuneetPunamiya @vdemeester