Document vector_math Plane API

[faheemabbas766]

Summary

• Documents the public Plane API in vector_math and vector_math_64.
• Removes the file-level public_member_api_docs ignore for both Plane implementations.
• Adds a CHANGELOG.md entry for the docs update.

Part of flutter/flutter#186827.

Validation

• dart analyze
• dart test test/plane_test.dart
• dart pub global run flutter_plugin_tools analyze --packages=vector_math
• dart pub global run flutter_plugin_tools test --packages=vector_math
• dart pub global run flutter_plugin_tools format --packages=vector_math --fail-on-change --no-swift

[stuartmorgan-g]

I've updated the PR description to reflect that the fact that this PR only address two out of 24 references to that issue.

In the future, please don't indicate that a PR will close an issue if it only partially addresses on, as GitHub will auto-close the issue on PR submit with specific keywords that indicate that it's a fix.

[faheemabbas766]

I've updated the PR description to reflect that the fact that this PR only address two out of 24 references to that issue.

In the future, please don't indicate that a PR will close an issue if it only partially addresses on, as GitHub will auto-close the issue on PR submit with specific keywords that indicate that it's a fix.

Thanks for the clarification. Understood.

I’ll avoid using Fixes/Closes in future PRs when the change only partially addresses an issue, and will use wording like Part of instead.

[zanderso]

It looks like this PR is failing the dart format presubmit check.

[stuartmorgan-g]

Please see #22 (comment); this needs to use our contribution checklist.

[faheemabbas766]

Thanks for the guidance. I’ve submitted the vector_math documentation changes to the source repository here: https://github.com/google/vector_math.dart/pull/373\n\nI’ll leave this mirror PR as-is for now unless you prefer that I close it.

[zanderso]

Hi @faheemabbas766, sorry about the confusion and crossed wires on our side. The repo at https://github.com/google/vector_math.dart has been archived, and this repo (flutter/core-packages) is the correct target for your PR.

[Piinks]

LGTM

[Piinks]

Oh I forgot, this needs an update to the changelog and pubspec so we can publish as a new release. :)

[faheemabbas766]

Addressed, thanks! I bumped vector_math to 2.4.1 and added a 2.4.1 CHANGELOG entry for the public Plane API documentation.

I also ran the package validation, formatter check, analyzer, plane_test.dart, and publish check locally; all passed. The fork workflows are currently awaiting maintainer approval.

Resolved