Skip to content

ci: adding pr docs preview action#4173

Merged
germa89 merged 6 commits intomainfrom
ci/adding-pr-docs-preview-action
Aug 13, 2025
Merged

ci: adding pr docs preview action#4173
germa89 merged 6 commits intomainfrom
ci/adding-pr-docs-preview-action

Conversation

@germa89
Copy link
Collaborator

@germa89 germa89 commented Aug 13, 2025

Description

Deploy PR docs to a temporary directory in github-pages. A message in the PR indicates the link where to check the PR.

Issue linked

NA

Checklist

Copilot AI review requested due to automatic review settings August 13, 2025 09:54
@germa89 germa89 requested a review from a team as a code owner August 13, 2025 09:54
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 adds documentation deployment functionality for pull requests by introducing two new GitHub Actions workflows. The implementation allows PR documentation to be deployed to a temporary directory in the github-pages branch, with automatic cleanup when PRs are closed or merged.

  • Adds a new workflow for deploying PR documentation that triggers when docs build successfully
  • Implements automatic cleanup of PR documentation when PRs are closed or merged
  • Extends the main CI workflow to trigger on additional PR events needed for documentation deployment

Reviewed Changes

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

File Description
.github/workflows/pr-docs-cleaner.yml New workflow for cleaning up PR documentation after closure
.github/workflows/ci.yml Extended PR triggers and added doc deployment job

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@germa89 germa89 requested a review from moe-ad August 13, 2025 09:55
@github-actions github-actions bot added CI/CD Related with CICD, Github Actions, etc maintenance General maintenance of the repo (libraries, cicd, etc) labels Aug 13, 2025
@germa89
Copy link
Collaborator Author

germa89 commented Aug 13, 2025

I was wondering @moe-ad if having the cleanup in another workflow works with the current action implementation??

@github-actions
Copy link
Contributor

github-actions bot commented Aug 13, 2025

The documentation for this pull request will be available at https://mapdl.docs.pyansys.com/pull/4173. Please allow some time for the documentation to be deployed.

@codecov
Copy link

codecov bot commented Aug 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.28%. Comparing base (63a2755) to head (47bc089).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4173      +/-   ##
==========================================
- Coverage   91.29%   91.28%   -0.02%     
==========================================
  Files         193      193              
  Lines       15730    15730              
==========================================
- Hits        14361    14359       -2     
- Misses       1369     1371       +2     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@germa89
Copy link
Collaborator Author

germa89 commented Aug 13, 2025

#PeopleAreOnHolidays #MinorWorks LGTM.

@pyansys-ci-bot LGTM

Copy link
Contributor

@pyansys-ci-bot pyansys-ci-bot left a comment

Choose a reason for hiding this comment

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

✅ Approving this PR because germa89 said so in here 😬

LGTM

@germa89 germa89 merged commit 44d9719 into main Aug 13, 2025
81 of 82 checks passed
@germa89 germa89 deleted the ci/adding-pr-docs-preview-action branch August 13, 2025 15:14
@github-actions
Copy link
Contributor

This PR has been closed. Documentation for this pull request will shortly be removed from its former deployment address.

@germa89
Copy link
Collaborator Author

germa89 commented Aug 13, 2025

Action proving the deletion of the deployed docs: https://github.com/ansys/pymapdl/actions/runs/16941487742

Congratulations on this @moe-ad, it works great!!

GitHub
A Python client library for Ansys MAPDL. Contribute to ansys/pymapdl development by creating an account on GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Related with CICD, Github Actions, etc maintenance General maintenance of the repo (libraries, cicd, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants