Skip to content

fix: Don't run the GitHub Action Deploy Dokka Docs on a PR.#257

Merged
mrudatsprint merged 1 commit intomainfrom
miker/issue-39
Dec 10, 2025
Merged

fix: Don't run the GitHub Action Deploy Dokka Docs on a PR.#257
mrudatsprint merged 1 commit intomainfrom
miker/issue-39

Conversation

@mrudatsprint
Copy link
Copy Markdown
Contributor

@mrudatsprint mrudatsprint requested a review from Copilot December 10, 2025 15:57
Copy link
Copy Markdown

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 modifies the GitHub Actions workflow to prevent the Dokka documentation deployment from running on pull requests, addressing issue #256. The workflow will now only execute on pushes to the main branch or through manual workflow dispatch.

Key Changes:

  • Removed the pull_request trigger from the Deploy Dokka Docs workflow

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@lyleschemmerling lyleschemmerling left a comment

Choose a reason for hiding this comment

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

makes sense

Copy link
Copy Markdown

@nathan-contino nathan-contino left a comment

Choose a reason for hiding this comment

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

LGTM, nice catch.

@mrudatsprint mrudatsprint marked this pull request as ready for review December 10, 2025 16:13
@mrudatsprint mrudatsprint requested review from a team as code owners December 10, 2025 16:13
@mrudatsprint mrudatsprint merged commit 9e3cabe into main Dec 10, 2025
20 of 21 checks passed
@mrudatsprint mrudatsprint deleted the miker/issue-39 branch December 10, 2025 16:13
@mrudatsprint mrudatsprint changed the title Don't run the GitHub Action Deploy Dokka Docs on a PR. fix: Don't run the GitHub Action Deploy Dokka Docs on a PR. Jan 22, 2026
This was referenced Jan 22, 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.

4 participants