Skip to content

ENH: Disable enablement - manually enable doc deployment#42

Merged
aylward merged 1 commit intoProject-MONAI:mainfrom
aylward:docdeploy
Apr 8, 2026
Merged

ENH: Disable enablement - manually enable doc deployment#42
aylward merged 1 commit intoProject-MONAI:mainfrom
aylward:docdeploy

Conversation

@aylward
Copy link
Copy Markdown
Collaborator

@aylward aylward commented Apr 7, 2026

Summary by CodeRabbit

  • Chores
    • Refined documentation deployment configuration to optimize the Pages setup process.

Copilot AI review requested due to automatic review settings April 7, 2026 22:27
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 54c59774-1ede-4665-a784-be16e7db0db4

📥 Commits

Reviewing files that changed from the base of the PR and between 4b38f45 and b17aabf.

📒 Files selected for processing (1)
  • .github/workflows/docs.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/docs.yml

Walkthrough

The documentation build workflow removes the enablement: true parameter from the Pages configuration step, simplifying the action invocation while preserving all other deployment workflow functionality.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/docs.yml
Removed with: enablement: true argument from actions/configure-pages@v4 in the "Setup Pages" step.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A config line hops away,
One true parameter at play,
Simpler steps now light the way,
Pages configure, hip-hooray! 🐰✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing the enablement parameter to allow manual control of doc deployment in the workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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 updates the GitHub Pages documentation deployment workflow by removing the explicit Pages “enablement” configuration from the actions/configure-pages step, aligning with a manual enablement approach for docs deployment.

Changes:

  • Remove the with: enablement: true input from actions/configure-pages@v4.

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

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@4b38f45). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #42   +/-   ##
=======================================
  Coverage        ?   13.77%           
=======================================
  Files           ?       49           
  Lines           ?     6605           
  Branches        ?        0           
=======================================
  Hits            ?      910           
  Misses          ?     5695           
  Partials        ?        0           
Flag Coverage Δ
integration-tests 13.77% <ø> (?)
unittests 13.77% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aylward aylward merged commit 782a7a4 into Project-MONAI:main Apr 8, 2026
18 checks passed
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