Skip to content

remove platform ci - #3840

Merged
Baalmart merged 1 commit into
stagingfrom
platform-cleanup
Jul 20, 2026
Merged

remove platform ci#3840
Baalmart merged 1 commit into
stagingfrom
platform-cleanup

Conversation

@Psalmz777

@Psalmz777 Psalmz777 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary of Changes (What does this PR do?)

  • Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Status of maturity (all need to be checked before merging):

  • I've tested this locally
  • I consider this code done
  • This change ready to hit production in its current state
  • The title of the PR states what changed and the related issues number (used for the release note).
  • I've included issue number in the "Closes #ISSUE-NUMBER" part of the "What are the relevant tickets?" section to link the issue.
  • I've updated corresponding documentation for the changes in this PR.
  • I have written unit and/or e2e tests for my change(s).

How should this be manually tested?

  • Please include the steps to be done inorder to setup and test this PR.

What are the relevant tickets?

Screenshots (optional)

Summary by CodeRabbit

  • Deployment Changes

    • Analytics Platform preview deployments are no longer built, deployed, or reported for staging pull requests.
    • Production and staging deployment workflows no longer include Analytics Platform deployments.
    • Analytics Platform preview environments are no longer removed by the automated preview cleanup process.
  • Quality Checks

    • Automated platform-specific linting, type checks, tests, and coverage uploads are no longer run through the removed CI workflows.

@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

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: 9edb2e93-e13b-42cd-922a-5e78fc1331a4

📥 Commits

Reviewing files that changed from the base of the PR and between c0092d1 and 97585eb.

📒 Files selected for processing (6)
  • .github/workflows/deploy-frontend-azurepreview.yml
  • .github/workflows/deploy-frontend-to-azureprod.yml
  • .github/workflows/deploy-frontend-to-azurestaging.yml
  • .github/workflows/platform-ci.yml
  • .github/workflows/platform-codecov.yml
  • .github/workflows/remove-deploy-azurepreview.yml
💤 Files with no reviewable changes (5)
  • .github/workflows/platform-ci.yml
  • .github/workflows/platform-codecov.yml
  • .github/workflows/deploy-frontend-to-azureprod.yml
  • .github/workflows/deploy-frontend-azurepreview.yml
  • .github/workflows/deploy-frontend-to-azurestaging.yml

📝 Walkthrough

Walkthrough

The changes remove analytics-platform preview and staging deployment jobs, their change-detection triggers, preview cleanup targeting, and dedicated platform CI and coverage workflows.

Changes

Analytics platform automation removal

Layer / File(s) Summary
Preview deployment removal
.github/workflows/deploy-frontend-azurepreview.yml, .github/workflows/remove-deploy-azurepreview.yml
Preview detection, deployment, readiness checks, PR comments, and analytics-p cleanup are removed.
Staging deployment and CI removal
.github/workflows/deploy-frontend-to-azurestaging.yml, .github/workflows/platform-ci.yml, .github/workflows/platform-codecov.yml
Staging analytics-platform triggering and deployment are removed along with the platform lint, type-check, test, coverage, and Codecov workflows.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

Suggested reviewers: baalmart

Poem

Platform winds grow still,
Preview lights fade from the sky,
Staging paths unwind.
CI checks close their books,
Azure keeps the rest in flight.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly reflects the main change: removing platform CI-related workflows and jobs.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch platform-cleanup

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)
  • JIRA integration encountered authorization issues. Please disconnect and reconnect the integration in the CodeRabbit UI.

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.

@Psalmz777 Psalmz777 self-assigned this Jul 20, 2026
@Baalmart
Baalmart merged commit 6db9c8c into staging Jul 20, 2026
26 checks passed
@Baalmart
Baalmart deleted the platform-cleanup branch July 20, 2026 09:05
@Baalmart Baalmart mentioned this pull request Jul 20, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants