Commit 073e07e
Fix InitRxUI to account for order of passed registration namespaces (#3887)
<!-- Please be sure to read the
[Contribute](https://github.com/reactiveui/reactiveui#contribute)
section of the README -->
**What kind of change does this PR introduce?**
<!-- Bug fix, feature, docs update, ... -->
Bug fix?
**What is the current behavior?**
<!-- You can also link to an open issue here. -->
Originally the `InitializeReactiveUI` function didn't account for order
of passed registration, because the allowed namespace dictionary was
enumerated instead of the array (which is kind of backwards if you think
about it).
**What is the new behavior?**
<!-- If this is a feature change -->
Order of the namespaces is preserved when registering.
**What might this PR break?**
Multi-platform projects that rely on current non-determinant order may
break because of different order of registrations.
**Please check if the PR fulfills these requirements**
- [ ] Tests for the changes have been added (for bug fixes / features)
- [ ] Docs have been added / updated (for bug fixes / features)
**Other information**:
Not sure on what kind of tests/docs are expected, so I am leaving it as
draft for now, feel free to undraft if you feel this is OK.
Co-authored-by: Glenn <[email protected]>1 parent 9dbac6e commit 073e07e
1 file changed
+3
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
58 | 58 | | |
59 | 59 | | |
60 | 60 | | |
61 | | - | |
62 | | - | |
63 | | - | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
64 | 64 | | |
65 | 65 | | |
66 | 66 | | |
| |||
0 commit comments