Skip to content

Rename dart repo - #575

Merged
beeme1mr merged 2 commits into
mainfrom
beeme1mr-patch-1
Sep 15, 2026
Merged

beeme1mr merged 2 commits into
mainfrom
beeme1mr-patch-1

Conversation

@beeme1mr

Copy link
Copy Markdown
Member

Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 894bca92-f152-4190-aedf-6930750ba083

📥 Commits

Reviewing files that changed from the base of the PR and between 7736e84 and 8d8a12b.

📒 Files selected for processing (2)
  • config/open-feature/org.yaml
  • config/open-feature/sdk-dart/workgroup.yaml

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

The organization and Dart workgroup configurations now reference dart-sdk instead of dart-server-sdk.

Changes

Dart repository configuration

Layer / File(s) Summary
Update Dart repository references
config/open-feature/org.yaml, config/open-feature/sdk-dart/workgroup.yaml
The repository entry changes from dart-server-sdk to dart-sdk in both configuration files.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Suggested reviewers: toddbaert, thomaspoignant

Merge Risk: ⚪ Minimal · up to 8d8a1

This configuration-only rename is internally consistent and presents no identified merge-blocking risk.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the repository rename from dart-server-sdk to dart-sdk.
Description check ✅ Passed The description links to the discussion related to the repository rename, so it is relevant to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.

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

Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
@beeme1mr
beeme1mr deployed to peribolos-verify September 14, 2026 19:57 — with GitHub Actions Active
@beeme1mr
beeme1mr marked this pull request as ready for review September 15, 2026 17:54
@beeme1mr
beeme1mr requested review from a team as code owners September 15, 2026 17:54
@beeme1mr
beeme1mr merged commit f243f25 into main Sep 15, 2026
3 checks passed
@beeme1mr
beeme1mr deleted the beeme1mr-patch-1 branch September 15, 2026 17:58

This branch was successfully deployed

1 active deployment
peribolos-verify — 8d8a12bd Deployed Sep 14, 2026 by beeme1mr via build #2088
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