@hideConstantImplementations by default.

[kallentu]

@hideConstantImplementations is now removed and applied by default where it was checked before.
Updated the tests and moved them to the constant_values_test.dart file.
Removed that directive information on the README.

Bug: #3564.

---

• I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

• See our contributor guide for general expectations for PRs.
• Larger or significant changes should be discussed in an issue before creating a PR.
• Contributions to our repos should follow the Dart style guide and use dart format.
• Most changes should add an entry to the changelog and may need to rev the pubspec package version.
• Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

[srawlins]

Nice!

[srawlins]

Flutter has a check for non-replaced doc directives, as seen in the CI. I think we can just remove that directive from flutter, and then land this, and then cut a release.

[kallentu]

Sounds good, I'm on it.

[kallentu]

@srawlins Okay, took a while to figure out what flutter needs to pass lol. The PR currently parses and hides the hideConstantImplementations directive, but it's not useful.
PTAL and I'll submit, cut a release, make a Flutter PR removing the directive usages + bumping the version and come back to delete this directive fully.

[srawlins]

Nice! This is perfect.