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.

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

Use v1 Tekton API #75

Merged
merged 1 commit into from
Oct 26, 2023
Merged

Use v1 Tekton API #75

merged 1 commit into from
Oct 26, 2023

Conversation

zregvart
Copy link
Member

ref: EC-29

@simonbaird
Copy link
Member

I thought these were updated automatically through some CI, possibly https://github.com/enterprise-contract/tekton-catalog/actions/workflows/sync-ec-cli-tasks.yaml but now that I look at it, perhaps it is just going to bump the digests.

@zregvart zregvart merged commit 16b3657 into main Oct 26, 2023
1 check passed
@zregvart zregvart deleted the issue/EC-29 branch October 26, 2023 08:18
@zregvart
Copy link
Member Author

Yeah, that was my thought initially, we only update the digests. (Mental note, if we change the Task definitions we need to update here as well)

@lcarva
Copy link
Member

lcarva commented Oct 26, 2023

Hmm.. it should be copying everything over:

cp -r "${EC_CLI_REPO_PATH}/tasks" .

Basically, no manual changes should be done against the task directory. At least that was the intention. Maybe something is broken?

@zregvart
Copy link
Member Author

Most likely I looked at it too quickly and failed to see what is done...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants