Skip to content

Conversation

@SamErde
Copy link
Collaborator

@SamErde SamErde commented Mar 27, 2025

This pull request includes updates to the .github/workflows/Deploy MkDocs.yml file to improve the deployment process for MkDocs. The most important changes include specifying paths to trigger the workflow and adding permissions and concurrency settings.

Workflow improvements:

  • .github/workflows/Deploy MkDocs.yml: Added paths to trigger the deployment workflow only when changes occur in the Docs folder, the workflow file itself, or the mkdocs.yml file.
  • .github/workflows/Deploy MkDocs.yml: Added write permissions for contents and set up concurrency settings to ensure that only one deployment runs at a time, canceling any in-progress deployments if a new one is triggered.

# Optimize and Steps to Reduce Failures

## Changes

- Define scoped paths to trigger workflow only if docs, the MkDocs config, or the workflow itself get updated.
- Explicitly defined write permissions for contents.
- Added concurrency setting to stop if a new instance of the workflow starts.
@SamErde SamErde added documentation Improvements or additions to documentation enhancement New feature or request labels Mar 27, 2025
@SamErde SamErde requested review from Copilot and jakehildreth March 27, 2025 01:34
@SamErde SamErde self-assigned this Mar 27, 2025
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 MkDocs deployment workflow to improve when and how the workflow is triggered as well as to better manage deployment concurrency.

  • Trigger the workflow only when specific files/folders change.
  • Add write permissions and concurrency settings to ensure that only one deployment runs at a time.
Comments suppressed due to low confidence (1)

.github/workflows/Deploy MkDocs.yml:13

  • If 'mkdocs.yml' is in the repository root, the leading './' prefix might prevent the pattern from matching as expected. Consider removing the prefix to ensure consistent path matching with the other entries.
- './mkdocs.yml'

@SamErde SamErde merged commit 550e9fe into main Mar 27, 2025
7 checks passed
@SamErde SamErde deleted the chore-Improve-MkDocs-workflow branch March 27, 2025 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants