Skip to content

Revert "fix(ci): preserve PR preview routes during release deployment"#223

Merged
sudip-mondal-2002 merged 1 commit into
mainfrom
revert-222-fix-ci-preview-routes
May 24, 2026
Merged

Revert "fix(ci): preserve PR preview routes during release deployment"#223
sudip-mondal-2002 merged 1 commit into
mainfrom
revert-222-fix-ci-preview-routes

Conversation

@sudip-mondal-2002

@sudip-mondal-2002 sudip-mondal-2002 commented May 24, 2026

Copy link
Copy Markdown
Owner

Reverts #222

Summary by CodeRabbit

  • Chores
    • Updated GitHub Pages deployment workflow configuration to adjust file handling during website publishing.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 24, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e1687841-82dd-4ed6-a73c-033d27ce306f

📥 Commits

Reviewing files that changed from the base of the PR and between 6c39508 and 0bf37d8.

📒 Files selected for processing (1)
  • .github/workflows/release.yml

📝 Walkthrough

Walkthrough

The release workflow's GitHub Pages deployment step removes the keep_files: true option from the peaceiris/actions-gh-pages@v4 action configuration. This simplifies the publication settings to only specify the token, publish directory (./docs), and target branch (gh-pages).

Changes

GitHub Pages Deployment Configuration

Layer / File(s) Summary
Remove keep_files from GitHub Pages publish step
.github/workflows/release.yml
The peaceiris/actions-gh-pages@v4 action configuration removes keep_files: true, retaining only the GitHub token, publish directory, and publish branch settings.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • sudip-mondal-2002/Amplitron#222: Modifies the same peaceiris/actions-gh-pages@v4 step in the release workflow by changing the keep_files configuration setting.

Poem

🐰 A workflow hops with lighter feet,
keep_files removed, the config's neat,
No ancient files clog the deploy,
Fresh pages published—a rabbit's joy! 🌿

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch revert-222-fix-ci-preview-routes

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.

@sudip-mondal-2002 sudip-mondal-2002 merged commit 887d379 into main May 24, 2026
6 of 7 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Code Coverage Report 📊

Line Coverage: 70.3%

✅ Coverage meets threshold: 70.3% >= 60%

Full Coverage Summary
Summary coverage rate:
  lines......: 70.3% (2962 of 4215 lines)
  functions..: 85.0% (441 of 519 functions)
  branches...: no data found

@sudip-mondal-2002 sudip-mondal-2002 added type:bug Something isn't working type:docs Improvements or additions to documentation type:feature New feature or request gssoc:approved GSSoC 2026 contribution type:performance Performance optimization or latency improvement type:testing Unit tests, integration tests, or test coverage type:security Security vulnerability or hardening level:critical quality:exceptional type:accessibility type:design type:refactor type:devops labels Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved GSSoC 2026 contribution level:critical quality:exceptional type:accessibility type:bug Something isn't working type:design type:devops type:docs Improvements or additions to documentation type:feature New feature or request type:performance Performance optimization or latency improvement type:refactor type:security Security vulnerability or hardening type:testing Unit tests, integration tests, or test coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant