Skip to content

Conversation

@chriswblake
Copy link
Member

This pull request introduces significant improvements to the GitHub Copilot code review exercise, focusing on clarity, usability, and enhanced automation. The changes update instructional content for better user guidance, add more realistic screenshots, refine workflow automation for step validation, and improve development environment setup and dependency installation.

Instructional Content and User Experience Improvements:

  • Updated .github/steps/1-step.md, .github/steps/2-step.md, .github/steps/3-step.md, and .github/steps/4-step.md to provide clearer instructions, reorganize activities, add new and more accurate screenshots, and clarify the use of Copilot reviews and custom instructions. This includes improved explanations of local code review, repository instructions, and ruleset enforcement, as well as more actionable step-by-step activities. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15]

Workflow Automation Enhancements:

  • Added a new check_step_work job to .github/workflows/1-step.yml to automatically verify that key changes (such as the announcement banner) are present in the correct files. This includes step result reporting and job failure on missing changes, improving feedback and validation for exercise participants.
  • Updated .github/workflows/2-step.yml to use more robust logic for checking if Copilot was assigned as a reviewer, ensuring accurate automation in step validation.

Development Environment and Dependency Improvements:

  • Enhanced .devcontainer/devcontainer.json by adding the GitHub CLI feature for improved developer tooling in Codespaces.
  • Improved MongoDB installation in .devcontainer/installMongoDB.sh by switching from deprecated apt-key to gpg --dearmor, ensuring compatibility and security with modern Ubuntu systems.

@chriswblake chriswblake self-assigned this Sep 9, 2025
@chriswblake chriswblake merged commit 0d74622 into main Sep 9, 2025
1 check passed
@chriswblake chriswblake deleted the cwb-test-run-adjustments branch September 9, 2025 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants