Skip to content

Conversation

@FidelusAleksander
Copy link
Contributor

Changes

This will be automated in the future

Checklist

  • I have added or updated appropriate labels to this PR
  • I have tested my changes
  • I have updated the documentation if needed

Copilot AI review requested due to automatic review settings April 23, 2025 16:23
@github-actions github-actions bot added the workflows Changes to reusable workflows label Apr 23, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates self-references in GitHub workflows to use the latest commit hash for the exercise toolkit.

  • Updated commit references in .github/workflows/start-exercise.yml
  • Updated commit references in .github/workflows/finish-exercise.yml

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/start-exercise.yml Updated exercise toolkit references to latest commit
.github/workflows/finish-exercise.yml Updated exercise toolkit references to latest commit
Comments suppressed due to low confidence (4)

.github/workflows/start-exercise.yml:70

  • [nitpick] Consider using the defined environment variable EXERCISE_TOOLKIT_REF here (e.g., ref: "${{ env.EXERCISE_TOOLKIT_REF }}") to avoid hardcoding the commit hash.
ref: 7aa247792cb4021203b7c05e65f9128d1c7d2312

.github/workflows/start-exercise.yml:113

  • [nitpick] Consider using the environment variable EXERCISE_TOOLKIT_REF here instead of hardcoding the commit hash to ensure consistency across the workflow.
ref: 7aa247792cb4021203b7c05e65f9128d1c7d2312

.github/workflows/finish-exercise.yml:31

  • [nitpick] Consider reusing the environment variable EXERCISE_TOOLKIT_REF here (e.g., ref: "${{ env.EXERCISE_TOOLKIT_REF }}") to promote DRY principles.
ref: 7aa247792cb4021203b7c05e65f9128d1c7d2312

.github/workflows/finish-exercise.yml:64

  • [nitpick] Consider using the environment variable EXERCISE_TOOLKIT_REF here as well, instead of hardcoding, to maintain consistency and ease future updates.
ref: 7aa247792cb4021203b7c05e65f9128d1c7d2312

@FidelusAleksander FidelusAleksander added the skip-release-notes Pull requests that should not be included in the release notes label Apr 23, 2025
@FidelusAleksander FidelusAleksander merged commit eb38f0d into main Apr 23, 2025
6 checks passed
@FidelusAleksander FidelusAleksander deleted the update-self-refs branch April 23, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-release-notes Pull requests that should not be included in the release notes workflows Changes to reusable workflows

Development

Successfully merging this pull request may close these issues.

2 participants