Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

wolz-CODElife
Copy link

Summary

This PR enhances the connect-the-dots skills exercise by converting it to an Issue-based flow, ensuring dynamic issue references, and improving workflow reliability and modularity. The changes support a scalable learner experience with better debugging and progression tracking.

Changes

  • Set write permissions for contents, actions, and issues to enable workflow operations.
  • Defined the template files(STEP_1_FILE, STEP_2_FILE, ...,STEP_n_FILE) as environment variable for consistent template referencing.
  • Implemented start_exercise job using skills/exercise-toolkit/start-exercise.yml@v0.3.0 to initialize the exercise and output the issue URL.
  • Added post_next_step_content job to:
    • Create dynamic issues ("Fix the broken sidebar" and "Sidebar bug again") using actions/github-script@v7.
    • Prepare file history with _sidebar.md updates and commit SHA storage.
    • Build and post content with action-text-variables@v1, using {{original_issue}} and {{duplicate_issue}} for dynamic referencing.
    • Made issue identifiers {{original_issue}} and {{duplicate_issue}} dynamic, replacing hardcoded values to support multiple concurrent issue creations.
    • Change the navigation system to use: gh workflow enable
  • Added find_exercise job to locate the exercise issue using skills/exercise-toolkit.
  • Added check_step_work and verify_step_work jobs to:
    • Post "checking work" feedback and verify learner progress.
    • Use actions/github-script@v7 for context debugging and output tracking.
  • Use new step grading table
  • Enhanced post_next_step_content with dynamic content and workflow toggling.

Closes: skills#94

Task list

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

…low logic

feat: add step to create comment with updated issue content

fix: remove unnecessary GH_TOKEN variable from comment creation step

fix: update comment creation step to use ISSUE_BODY variable for improved clarity

fix: correct syntax for template-vars in comment creation step

fix: streamline PR validation and disable workflow step

fix: update response templates step and improve branch name check

fix: enhance comment updates and error handling in workflow steps
…kflow steps

fix: update commit validation steps and streamline PR checks

fix: enhance validation steps for issue comments and streamline workflow dependencies
…ow checks

fix: correct indentation for check_step_work job in workflow files

fix: reorder checks for PR body and merge state in sidebar workflow

fix: enhance failure condition to include PR merge state in workflow
@wolz-CODElife wolz-CODElife merged commit 6d8a80d into skills-dev-2:main Jun 27, 2025
5 checks passed
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.

Migrate exercise to issue-based flow

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.