Skip to content

ReactivePropertySlimに対応 - #239

Merged
neuecc merged 3 commits into
MessagePack-CSharp:masterfrom
reniris:SlimResolver
Apr 26, 2018
Merged

ReactivePropertySlimに対応#239
neuecc merged 3 commits into
MessagePack-CSharp:masterfrom
reniris:SlimResolver

Conversation

@reniris

@reniris reniris commented Apr 24, 2018

Copy link
Copy Markdown

ReactivePropertySlimResolverを追加
ReactivePropertyのバージョンアップにより、.NET4.5のサポートを切って.NET4.6をサポート。
これでReactivePropertyも正常にシリアライズ、デシリアライズできます。

@neuecc

neuecc commented Apr 25, 2018

Copy link
Copy Markdown
Member

ありがとうございます、良いと思います!
ただ、Resolverは分ける必要がないと思うので、↓のあたりに追加して統合していただければです。
https://github.com/reniris/MessagePack-CSharp/blob/80d3bf2d53dd396aeab44279efdee6d8d812fd31/src/MessagePack.ReactiveProperty/ReactivePropertyResolver.cs#L37-L43

@reniris

reniris commented Apr 25, 2018

Copy link
Copy Markdown
Author

わかりました。しばらくお待ちください。

@reniris

reniris commented Apr 25, 2018

Copy link
Copy Markdown
Author

Resolverを統合してSlimResolverブランチを更新しました。
IReactivePropertyは何もしていません。

@reniris

reniris commented Apr 25, 2018

Copy link
Copy Markdown
Author

新しくIFRPropブランチを切って、IReactivePropertyのシリアライズ、デシリアライズをReactivePropertySlimに対応させました。
https://github.com/reniris/MessagePack-CSharp/tree/IFRProp

@neuecc

neuecc commented Apr 25, 2018

Copy link
Copy Markdown
Member

どうもです、OKかとー。
Interfaceのもなるほどそうですね。
このPRで両対応にしちゃっていいと思うので混ぜちゃってください。
そうしたらマージします。

@reniris

reniris commented Apr 26, 2018

Copy link
Copy Markdown
Author

マージしました。

@neuecc
neuecc merged commit 9a7fa63 into MessagePack-CSharp:master Apr 26, 2018
@neuecc

neuecc commented Apr 26, 2018

Copy link
Copy Markdown
Member

どうもです、マージしました!
リリースは、ちょっと幾つか溜まってる必須Issueを片付けた時と同タイミングを考えています。

AArnott pushed a commit that referenced this pull request Jan 16, 2024
Bumps [xunit](https://github.com/xunit/xunit) from 2.6.2 to 2.6.3.
- [Commits](xunit/xunit@2.6.2...2.6.3)

---
updated-dependencies:
- dependency-name: xunit
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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