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

Camunda 24303 fix action path #318

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

Kerruba
Copy link
Contributor

@Kerruba Kerruba commented Dec 19, 2024

This PR is for implementing the cache disable mechanism but in a way that actually works when referencing the is-cache-enabled composite action

@Kerruba Kerruba requested a review from cmur2 December 19, 2024 14:03
Comment on lines +41 to +45
- name: Symlink current action repo
env:
action_path: ${{ github.action_path }}
run: ln -fs ${{ env.action_path }}/.. .github/.linked-infra-global-github-actions
shell: bash
Copy link
Member

Choose a reason for hiding this comment

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

This workaround is necessary as per https://github.com/orgs/community/discussions/41927

run: |
IS_PR_EVENT=${{ github.event_name == 'pull_request' }}
HAS_NO_CACHE_LABEL=${{ contains(github.event.pull_request.labels.*.name, 'ci:no-cache') }}
IS_FEATURE_BRANCH=${{ !(startsWith(github.ref_name, 'stable/') || github.ref_name == 'main') }}
Copy link
Member

Choose a reason for hiding this comment

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

Introduce and respect the cache_create_branch_regex input that setup-yarn-cache action has - to allow universal use also outside of camunda/camunda repo.

Kerruba added a commit that referenced this pull request Dec 20, 2024
I've moved the setup-yarn-cache code (and its dependency on
is-cache-enabled) to a dedicated PR.
This commit has been used also for testing on the monorepo
- camunda/camunda#26294

This will replace:
- #318
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