Skip to content

Conversation

@ncipollina
Copy link
Contributor

🚀 Pull Request

📋 Summary

Briefly describe what this PR does and why it's needed.


✅ Checklist

  • My changes build cleanly
  • I’ve added/updated relevant tests
  • I’ve added/updated documentation or README
  • I’ve followed the coding style for this project
  • I’ve tested the changes locally (if applicable)

🧪 Related Issues or PRs

Closes #...


💬 Notes for Reviewers

Any specific areas to look at, known issues, or follow-up work.

@ncipollina ncipollina requested a review from Copilot July 11, 2025 00:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the GitHub Actions workflow for documentation deployment by replacing the existing MkDocs workflow with a new, more comprehensive version. The new workflow includes improved dependency management, caching, and conditional deployment logic.

Key changes:

  • Replaces the old mkdocs-material-gh-pages.yml with a new docs.yml workflow
  • Adds pull request triggers for documentation validation
  • Implements dependency caching and conditional deployment to production

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
LayeredCraft.Cdk.Constructs.sln Updates solution file reference to point to the new workflow filename
.github/workflows/mkdocs-material-gh-pages.yml Removes the old documentation deployment workflow
.github/workflows/docs.yml Adds new documentation workflow with enhanced features including PR validation, caching, and conditional deployment

@ncipollina ncipollina merged commit 7cbf329 into main Jul 11, 2025
3 checks passed
@ncipollina ncipollina deleted the feature/refactor-docs branch July 11, 2025 00:31
@ncipollina
Copy link
Contributor Author

@all-contributors please add @ncipollina for code, doc, and infrastructure

@allcontributors
Copy link
Contributor

@ncipollina

I've put up a pull request to add @ncipollina! 🎉

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