Refactor devcontainer setup and improve README and testing#15
Open
Tdblackston0 wants to merge 15 commits into
Open
Refactor devcontainer setup and improve README and testing#15Tdblackston0 wants to merge 15 commits into
Tdblackston0 wants to merge 15 commits into
Conversation
…guration - Simplified devcontainer.json by removing unnecessary Docker Compose settings and added remote environment variables. - Updated README to clarify prerequisites and setup instructions for various environments. - Improved CartContext tests by adding missing imports and adjusted vite.config to use vitest/config. - Created post-create.sh script for automated dependency installation and project setup. - Cleaned up package-lock.json files by removing unnecessary peer dependencies.
Refactor devcontainer setup, update README, and enhance testing confi…
Update README: add manual path prerequisites and installation instruc…
Author
|
@cherryjain-msft - Take a look at my updates when you have a chance please. |
Agent-Logs-Url: https://github.com/Tdblackston0/Dev-Toil-Reduction/sessions/964479fe-a4f2-4a32-848c-dddc6edeb602 Co-authored-by: Tdblackston0 <6003490+Tdblackston0@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Tdblackston0/Dev-Toil-Reduction/sessions/964479fe-a4f2-4a32-848c-dddc6edeb602 Co-authored-by: Tdblackston0 <6003490+Tdblackston0@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Tdblackston0/Dev-Toil-Reduction/sessions/64cd7da4-6372-4e3a-af61-bffb663c8886 Co-authored-by: Tdblackston0 <6003490+Tdblackston0@users.noreply.github.com>
This reverts commit d09e81b. Co-authored-by: Tdblackston0 <6003490+Tdblackston0@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Tdblackston0/Dev-Toil-Reduction/sessions/41387aee-8242-4ad1-ac81-f766f8ad2bee Co-authored-by: Tdblackston0 <6003490+Tdblackston0@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Tdblackston0/Dev-Toil-Reduction/sessions/96e9845b-87a5-4ea4-bad4-61da1b62588d Co-authored-by: Tdblackston0 <6003490+Tdblackston0@users.noreply.github.com>
…ub MCP server configuration
…line-endings Copilot/fix devcontainer and line endings
- Bump GitHub Actions to latest versions (checkout v5→v6, setup-node v5→v6, github-script v7→v8) - Fix branch references from 'main' to 'master' in Labs 01 and 06 - Remove redundant npm run db:seed:dev from Lab 01 Step 8 - Add PowerShell alternative for curl commands in Lab 01 Step 9 - Add cd api instruction to Lab 01 Step 10 for running tests Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
fix: workshop lab accuracy improvements
Added troubleshooting tips for Podman and CA certification requirements.
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.
This pull request significantly improves the developer onboarding experience and streamlines the development environment setup for the project. The main changes include a complete overhaul of the
.devcontainerconfiguration, a major rewrite of theREADME.mdto clarify setup paths and troubleshooting, and the addition of a post-create script to automate dependency installation and build steps. There are also minor package lockfile adjustments.Development Environment Overhaul
.devcontainer/devcontainer.jsonto use a prebuilt image (mcr.microsoft.com/devcontainers/typescript-node:24-bookworm), simplified features, and added explicit environment variables and resource requirements. This reduces setup time and complexity, especially for Codespaces and Dev Containers. [1] [2]post-create.shscript to automate dependency installation and build steps after container creation, with clear next-step instructions for authentication and running the app..vscode/settings.json)Documentation Improvements
Dependency and Lockfile Maintenance
package-lock.jsonfiles in bothapiandfrontend, mainly removing or adjusting"peer": trueflags to reduce potential npm warnings and ensure consistency. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18]These changes will make it much easier for new contributors and workshop participants to get up and running quickly, with fewer manual steps and less troubleshooting required.