Skip to content

Conversation

@reniris
Copy link

@reniris reniris commented Apr 24, 2018

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

@neuecc
Copy link
Member

neuecc commented Apr 25, 2018

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

@reniris
Copy link
Author

reniris commented Apr 25, 2018

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

@reniris
Copy link
Author

reniris commented Apr 25, 2018

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

@reniris
Copy link
Author

reniris commented Apr 25, 2018

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

@neuecc
Copy link
Member

neuecc commented Apr 25, 2018

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

@reniris
Copy link
Author

reniris commented Apr 26, 2018

マージしました。

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

neuecc commented Apr 26, 2018

どうもです、マージしました!
リリースは、ちょっと幾つか溜まってる必須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] <[email protected]>
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