Migrate to issue based exercise flow#60
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the exercise workflow templates to a new issue-based flow, updating the step and workflow files while removing outdated templates.
- Removed old step workflows (0-tbd, 1-tbd, 2-tbd, 3-tbd, 0-welcome)
- Added new step workflows and updated exercise step markdown files
- Updated the README and other templates with new placeholder tags
Reviewed Changes
Copilot reviewed 23 out of 23 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/3-tbd.yml | Removed outdated workflow for Step 3 |
| .github/workflows/2-tbd.yml | Removed outdated workflow for Step 2 |
| .github/workflows/1-tbd.yml | Removed outdated workflow for Step 1 |
| .github/workflows/0-welcome.yml | Removed welcome workflow as part of the new flow |
| .github/workflows/2-step.yml | Added new step workflow with issue comment automation |
| .github/workflows/1-step.yml | Added new step workflow for Step 1 |
| .github/workflows/0-start-exercise.yml | Added a workflow to kick off the exercise |
| .github/steps/* | Updated and added step markdown files for the new exercise flow |
| .github/dependabot.yml | Removed Dependabot configuration |
Comments suppressed due to low confidence (2)
.github/workflows/1-step.yml:20
- The environment variable is named STEP_2_FILE in a workflow that appears to correspond to Step 1. Consider renaming it to STEP_1_FILE for clarity.
STEP_2_FILE: ".github/steps/1-step.md"
.github/dependabot.yml:1
- [nitpick] Review whether the removal of the Dependabot configuration for GitHub Actions updates is intentional, as automated dependency updates can help maintain workflow security and functionality.
Removed dependabot configuration
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
All new exercises we create for skills.github.com follow a new flow.
This PR updates the template to support that issue based flow
There are series of references "replace_me" that can be found by
CTRL+Fso you remember all the things you need to replace when creating a new exercise. They are also there to instruct Copilot to find these parts and update when provided an exercise outline documentChanges
Replaced all step and workflow files to match new flow
Updated README
Closes:
Task list