Skip to content

Conversation

@mtrezza
Copy link
Member

@mtrezza mtrezza commented Nov 29, 2025

Summary by CodeRabbit

  • Chores
    • Reorganized CI/CD publishing workflow to consolidate logic and improve maintainability.
    • Enhanced pipeline modularity for coordinating Dart and Flutter package releases and documentation deployment.

✏️ Tip: You can customize this high-level summary in your review settings.

@parse-github-assistant
Copy link

🚀 Thanks for opening this pull request!

@coderabbitai
Copy link

coderabbitai bot commented Nov 29, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The release CI workflow was refactored to expose semantic-release tag outputs and consolidate publishing logic. The main release job now outputs new_release_git_tag for Dart and Flutter. Publishing logic was extracted into a new external workflow (release-publish.yml) that the main release workflow delegates to conditionally based on tag presence, replacing the inline publish steps.

Changes

Cohort / File(s) Summary
Release workflow restructuring
.github/workflows/release-automated.yml
Modified to add step-level ids (dart-release, flutter-release) and expose new_release_git_tag outputs from semantic-release steps. Replaced inline pub-publish and docs-publish jobs with a single publish job that conditionally runs when dart_tag or flutter_tag is non-empty and calls external workflow release-publish.yml.
New publishing workflow
.github/workflows/release-publish.yml
New workflow file that orchestrates Dart and Flutter package publishing and documentation deployment. Supports workflow_dispatch and workflow_call triggers. Defines multi-step jobs for pub-publish and docs-publish for both Dart and Flutter, including checkout, SDK setup, dependency installation, linting, dry-run, and publish. Docs jobs generate and deploy to gh-pages branch. Includes conditional execution based on tag prefixes and job-level timeouts.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20–25 minutes

  • Output/input mapping verification: Confirm that semantic-release step outputs (new_release_git_tag) are correctly exposed from the release job and properly received by the publish job as dart_tag and flutter_tag inputs.
  • Conditional execution logic: Verify the publish job's condition (if: needs.release.outputs.dart_tag != '' || needs.release.outputs.flutter_tag != '') and conditional execution within release-publish.yml jobs based on tag presence/prefixes.
  • Workflow call inputs: Ensure all inputs passed to the external release-publish.yml workflow are correctly named and formatted.
  • Job dependencies and ordering: Confirm that release-publish.yml jobs are ordered correctly and wait for dependencies before execution.
  • Permission scopes: Validate that id-token permission is sufficient for pub.dev publishing and gh-pages deployment.
  • Timeout values: Review job-level timeout settings to ensure they are appropriate for the respective publishing and documentation tasks.

Possibly related PRs

  • ci: Fix auto-release #1069: Modifies the release/publish CI flow including release/pub-publish job names and tag-based publish logic, indicating parallel or related restructuring of CI workflows.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 63b87a7 and 45960fd.

📒 Files selected for processing (2)
  • .github/workflows/release-automated.yml (2 hunks)
  • .github/workflows/release-publish.yml (1 hunks)

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.

@mtrezza mtrezza merged commit 5c26e5b into parse-community:master Nov 29, 2025
10 of 11 checks passed
@mtrezza mtrezza deleted the ci/fix-ar branch November 29, 2025 05:57
@codecov
Copy link

codecov bot commented Nov 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.46%. Comparing base (5b6aacf) to head (45960fd).
⚠️ Report is 43 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1070   +/-   ##
=======================================
  Coverage   43.46%   43.46%           
=======================================
  Files          61       61           
  Lines        3587     3587           
=======================================
  Hits         1559     1559           
  Misses       2028     2028           

☔ 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.

@parseplatformorg
Copy link

🎉 This change has been released in version 9.3.0

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