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

ci: Use refs for reusable workflows #375

Merged
merged 2 commits into from
Feb 3, 2025
Merged

ci: Use refs for reusable workflows #375

merged 2 commits into from
Feb 3, 2025

Conversation

ifrost
Copy link
Contributor

@ifrost ifrost commented Feb 3, 2025

✨ Description

Related issue(s): #295

Fixes incorrect paths / refs updated in the last commit of #364. It was supposed to use main branch workflows but:

  • the ref needs to be specified if owner/repo format is used
  • the ref can be skipped for workflows in the same repo but path needs to be relative

See docs in: https://docs.github.com/en/actions/sharing-automations/reusing-workflows#calling-a-reusable-workflow

📖 Summary of the changes

  • Use a relative path to a reusable workflow

🧪 How to test?

Workflows should be running for the PR (https://github.com/grafana/explore-profiles/actions/runs/13114010650)

@github-actions github-actions bot added the ci label Feb 3, 2025
@ifrost ifrost requested a review from grafakus February 3, 2025 12:58
Copy link
Contributor

github-actions bot commented Feb 3, 2025

Unit test coverage

Lines Statements Branches Functions
Coverage: 12%
13.1% (552/4213) 9.15% (137/1497) 9.42% (122/1294)

@grafakus grafakus merged commit a46559f into main Feb 3, 2025
12 checks passed
@grafakus grafakus deleted the ifrost/use-refs branch February 3, 2025 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants