Skip to content

Conversation

@EtienneM
Copy link
Member

No description provided.

@EtienneM EtienneM self-assigned this Nov 21, 2025
@notion-workspace
Copy link

@EtienneM EtienneM force-pushed the build/STORY-2981/dependabot-automerge branch from bffebf2 to 206df47 Compare November 24, 2025 07:40
@EtienneM EtienneM marked this pull request as ready for review November 24, 2025 07:41
@EtienneM EtienneM requested a review from a team as a code owner November 24, 2025 07:41
@EtienneM EtienneM requested review from matthieu526-scalingo and removed request for a team November 24, 2025 07:41
-
name: Checkout
- name: Checkout
if: startsWith(github.ref, 'refs/tags/')
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the purpose of this?

Copy link
Member Author

Choose a reason for hiding this comment

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

It skips the step if the push corresponds to the push of a Git tag (i.e. the Git ref starts with refs/tags/.

Copy link
Contributor

Choose a reason for hiding this comment

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

Could the release action also be in the shared repo?

Copy link
Member Author

Choose a reason for hiding this comment

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

In the long term, I think so. But for now each repository with a release step is doing the release its own way. Hence it requires more work. I don't want to invest time in this for now.

@EtienneM EtienneM merged commit 919778a into master Nov 25, 2025
7 checks passed
@EtienneM EtienneM deleted the build/STORY-2981/dependabot-automerge branch November 25, 2025 16:13
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