Skip to content

clean unnecessary copy #1133

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

Merged

Conversation

rsoaresd
Copy link
Contributor

@rsoaresd rsoaresd commented Mar 17, 2025

Copy link

openshift-ci bot commented Mar 19, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alexeykazakov, fbm3307, metlos, mfrancisc, rsoaresd, xcoulon

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:
  • OWNERS [alexeykazakov,fbm3307,metlos,mfrancisc,rsoaresd,xcoulon]

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

@MatousJobanek
Copy link
Collaborator

/retest
/lgtm

@MatousJobanek
Copy link
Collaborator

it's actually the same problem with pairing multiple PRs again. I didn't notice that :-) You either need to use different branch name or merge the one of the other PRs and delete the branch there first and then re-run this again.

@rsoaresd
Copy link
Contributor Author

@MatousJobanek i noticed here and on the was side. i will be more careful next time, I promise

@openshift-ci openshift-ci bot removed the lgtm label Mar 20, 2025
Copy link

@alexeykazakov
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Mar 20, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 63ca319 into codeready-toolchain:master Mar 21, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants