Skip to content

feat: Migrate exercise to issue based flow #270

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

Merged
merged 13 commits into from
May 28, 2025

Conversation

FidelusAleksander
Copy link
Contributor

Summary

Changes

  • Migrated to issue based flow (instead of updating README)
  • Added several grading checks

Closes:

Task list

  • For workflow changes, I have verified the Actions workflows function as expected.
  • For content changes, I have reviewed the style guide.

Copy link

@Copilot 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 migrates the exercise to an issue-based flow and adds automated grading checks for the learner’s welcome.yml workflow.

  • Introduces three new workflows (0-start-exercise.yml, 1-step.yml, 2-step.yml) that use the exercise-toolkit to manage steps via GitHub issues.
  • Adds keyphrase-based checks in steps 1 and 2 to validate welcome.yml contents before advancing.
  • Removes legacy README-based workflows (0-welcome.yml, 1-create-a-workflow.yml, 2-add-a-job.yml) in favor of issue-driven progression.

Reviewed Changes

Copilot reviewed 27 out of 27 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
.github/workflows/0-start-exercise.yml New start-exercise workflow using issues to launch the lesson
.github/workflows/1-step.yml Step 1 issue-based workflow with checks for welcome.yml setup
.github/workflows/2-step.yml Step 2 issue-based workflow with checks for job definitions
.github/workflows/0-welcome.yml Removed legacy welcome workflow
.github/workflows/1-create-a-workflow.yml Removed legacy create-workflow step
.github/workflows/2-add-a-job.yml Removed legacy add-job step

@chriswblake chriswblake changed the title Migrate the exercise to issue based flow feat: Migrate exercise to issue based flow May 28, 2025
@chriswblake chriswblake merged commit 323c050 into skills:main May 28, 2025
@skills skills locked as spam and limited conversation to collaborators May 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants