withBuildChild fix #1034

subzero911 · 2024-12-16T11:39:07Z

I fixed Observer.withBuiltChild constructor.
I removed runtime asserts and unneeded nullables. So now it's neat.

I also added an example to the docs.

Pull Request Checklist

If the changes are being made to code, ensure the version in pubspec.yaml is updated.
Increment the major/minor/patch/patch-count, depending on the complexity of change
Add the necessary unit tests to ensure the coverage does not drop
Update the Changelog to include all changes made in this PR, organized by version
Run the melos run set_version command from the root directory
Include the necessary reviewers for the PR
Update the docs if there are any API changes or additions to functionality

line len 80

Bumps [nanoid](https://github.com/ai/nanoid) from 3.3.7 to 3.3.8. - [Release notes](https://github.com/ai/nanoid/releases) - [Changelog](https://github.com/ai/nanoid/blob/main/CHANGELOG.md) - [Commits](ai/nanoid@3.3.7...3.3.8) --- updated-dependencies: - dependency-name: nanoid dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

subzero911 · 2024-12-16T11:59:47Z

Sooo.
I fixed analysis options (which were spoiled in #1033 )
I also bumped melos right here.
Let's do it all in a single PR to not to get confused.

Sergey Molchanovsky and others added 8 commits December 5, 2024 12:38

better withBuildChild

50b32be

line len 80

docs

386f17f

rename builderWithChild parameter to builder

0f30ad8

bump ver.

dda1b95

Merge branch 'main' into withBuildChild_fix

4a2b5d6

revert: ignore: unused_element

9e0f16e

bump melos

0185ad2

pavanpodila approved these changes Dec 16, 2024

View reviewed changes

pavanpodila merged commit d8ac087 into mobxjs:main Dec 16, 2024
2 of 7 checks passed