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

TECH-1140 Move Composite Actions into a separate #50

Merged

Conversation

tonisojandu-sympower
Copy link
Collaborator

This is to avoid self-reference release before the version commit exists and is tagged.
Self-reference would cause a Renovate loop in Sympower Actions.

Relative references are not supported yet:
actions/runner#1348
actions/runner#1684

This PR has been on the shelf since February. Comment from @nyarly from August suggests that this feature could compromise the feature set that is reserved for Enterprise users, so there is no confidence that it will be merged.

Tested the changes here:
https://github.com/sympower/msa-observability/actions/runs/6626012294
https://github.com/sympower/msa-observability/actions/runs/6626046945

…nce release Renovate loop in Sympower Actions
…f-reference release Renovate loop in Sympower Actions"

This reverts commit 87d8ab0.
…nce release Renovate loop in Sympower Actions
@florianbuth-sympower
Copy link
Collaborator

Can you clarify, what is the plan/schedule for removing them from here?

Copy link
Collaborator

@florianbuth-sympower florianbuth-sympower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Just had one question that I posted in a comment

@tonisojandu-sympower
Copy link
Collaborator Author

Can you clarify, what is the plan/schedule for removing them from here?

I wanted to remove them already, but yesterday I added a main branch reference to the composite actions as a workaround. Meaning, some may already refer to them transitively.

At the end of next week, it will be a new month, so all the teams will have received their Renovate PR. On 6th October I will go over and see, who are still the old versions or main references. I will then manually intervene in those cases.

@tonisojandu-sympower tonisojandu-sympower merged commit 00f890c into main Oct 24, 2023
@tonisojandu-sympower tonisojandu-sympower deleted the TECH-1140-move-composite-actions-into-different-repo branch October 24, 2023 12:15
@florianbuth-sympower
Copy link
Collaborator

Awesome, sounds good!

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.

2 participants