Skip to content

Merge main, resolve conflicts, and fix content path references#190

Draft
Copilot wants to merge 2 commits intosimplify-client-appfrom
copilot/sub-pr-179
Draft

Merge main, resolve conflicts, and fix content path references#190
Copilot wants to merge 2 commits intosimplify-client-appfrom
copilot/sub-pr-179

Conversation

Copy link
Contributor

Copilot AI commented Mar 9, 2026

PR #179 was in a dirty merge state against main. This merges main in, resolves conflicts, and updates all content paths to match the app/ directory restructure.

Conflict resolution

  • .gitignore: kept app/client/ and app/server/ paths; added .azure from main
  • README.md: updated to three workshops (added GitHub Actions link); kept updated tech stack description (Tailwind, no Svelte)

Content from main pulled in

  • Full content/github-actions/ workshop (11 files)
  • content/shared-images/ shared screenshot assets (moved from content/1-hour/images/)
  • next-steps.md

Path reference fixes (/clientapp/client, /serverapp/server)

  • content/1-hour/0-setup.mdscripts/start-app.shapp/scripts/start-app.sh
  • content/1-hour/1-add-endpoint.md — inline text and restart note
  • content/full-day/3-codespaces.mdpostStartCommand script path
  • content/github-actions/3-running-tests.md, 4-caching.md, 5-matrix-strategies.md, 7-custom-actions.md, 9-required-workflows.mdworking-directory, pip install -r, and cache-dependency-path references

Grammar fix

  • content/full-day/README.md: "The application for the workshop uses is built" → "The application used for the workshop is built"

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: GeekTrainer <6109729+GeekTrainer@users.noreply.github.com>
Copilot AI changed the title [WIP] Simplify client by removing Svelte and using pure Astro Merge main, resolve conflicts, and fix content path references Mar 9, 2026
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.

2 participants