Conversation
Preview DeploymentThis PR has been deployed for preview:
|
There was a problem hiding this comment.
Pull request overview
This PR introduces the initial version of a Docker training presentation built with reveal.js and Tailwind CSS. It follows the same architectural pattern as the existing cloud-migrations and secure-landing-zones presentations in the repository.
Changes:
- Adds the complete
docker-trainingpresentation directory with ~20 modular section files covering Docker fundamentals, container history, networking, volumes, Swarm, Kubernetes, and more. - Registers the new presentation in
scripts/build.jsandpackage.jsonso it is built and can be served alongside existing presentations.
Reviewed changes
Copilot reviewed 29 out of 75 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
scripts/build.js |
Adds docker-training to the PRESENTATIONS array so it is included in the build pipeline |
package.json |
Adds a start:docker-training npm script for local development serving |
docker-training/index.html |
Main HTML entry point; wires together all section files via data-external and initializes Reveal.js |
docker-training/tailwind.config.js |
Tailwind configuration scoped to the presentation's HTML/JS files |
docker-training/postcss.config.js |
PostCSS configuration enabling Tailwind and Autoprefixer |
docker-training/css/tailwind.css |
Tailwind CSS entry point with base/components/utilities directives |
docker-training/sections/*.html |
~20 modular section files comprising the slide content |
docker-training/plugin/external/external.js |
Plugin to load external HTML section files into reveal.js |
docker-training/plugin/title-footer/title-footer.{js,css} |
Optional title-footer plugin (bundled but not loaded in index.html) |
docker-training/imgs/*.png |
Binary image assets referenced by the slides |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Use HTTPS instead of HTTP for play-with-docker and awesome-docker links - Fix invalid GitHub URL for 56KCloud Training Resources - Rename duplicate "Docker Architecture" heading to "Docker Desktop Architecture" - Update obsolete ubuntu:15.04 to ubuntu:24.04 in Dockerfile example - Fix grammar: "Where and what do you work?" → "Where do you work and what do you do?" - Replace dead docker.com/what-container link with docs.docker.com/get-started/ https://claude.ai/code/session_014L2DHkoioyJHYgvdHB3Bwx
Address Copilot review feedback on docker training slides
Updated link to Docker container documentation.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 29 out of 75 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Add CODEOWNERS file with github-copilot[bot] as default owner
- Add docker-training presentation link to README.md - Rename duplicate "Training Agenda" to "Docker Labs Agenda" in docker-labs.html - Remove duplicate docker inspect entry from docker-commands.html Images section - Fix grammar: "VM's" → "VMs" (no apostrophe for plurals) in containers-vs-vms.html https://claude.ai/code/session_018yB5s12p5Q5Gme3jpJaaVZ Co-authored-by: Claude <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 31 out of 77 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 30 out of 76 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.