Skip to content

Upgrade guide - #476

Merged
andrewdalpino merged 4 commits into
3.0from
upgrade-guide
Sep 6, 2026
Merged

Upgrade guide#476
andrewdalpino merged 4 commits into
3.0from
upgrade-guide

Conversation

@andrewdalpino

Copy link
Copy Markdown
Member

No description provided.

@andrewdalpino
andrewdalpino requested review from a team and a lite review from Copilot September 6, 2026 00:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The new upgrade guide includes copy-pastable PHP examples and statements that are currently inconsistent with the library API defaults/signatures.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds a dedicated upgrade guide to the documentation site to help users migrate from Rubix ML 2.0 to 3.0, and links it from the MkDocs navigation.

Changes:

  • Added a new “Upgrading from 2.0 to 3.0” guide covering breaking changes, behavioral changes, and new features.
  • Linked the new guide from the “Getting Started” section in mkdocs.yml.
  • Removed a changelog bullet entry (“Emoji preset to Regex Filter”).
File summaries
File Description
mkdocs.yml Adds the Upgrade Guide page to the documentation navigation.
docs/upgrading-to-3.md New upgrade guide content for migrating from 2.0 → 3.0.
CHANGELOG.md Removes one bullet entry from the v3.0 changelog section.
Review details

Suppressed comments (1)

docs/upgrading-to-3.md:328

  • Pipeline's constructor expects the first argument to be an array of transformers, but this example passes a single transformer instance, which would throw a TypeError if copied verbatim.
// or inside a pipeline
$estimator = new Pipeline(new FloatTypeConverter(), new KMeans(5));
  • Files reviewed: 3/3 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/upgrading-to-3.md Outdated
Comment thread docs/upgrading-to-3.md Outdated
andrewdalpino and others added 2 commits September 5, 2026 19:12
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@andrewdalpino
andrewdalpino merged commit 9901c52 into 3.0 Sep 6, 2026
2 checks passed
@andrewdalpino
andrewdalpino deleted the upgrade-guide branch September 6, 2026 00:13
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