Skip to content

Conversation

@flagbug
Copy link
Collaborator

@flagbug flagbug commented Nov 30, 2015

The ReactiveUI logging infrastructure moved to Splat, so all the
documentation should go there. Since ReactiveUI defines some useful
logging helpers for observables, I've left the documentation of those in
there.

The ReactiveUI logging infrastructure moved to Splat, so all the
documentation should go there. Since ReactiveUI defines some useful
logging helpers for observables, I've left the documentation of those in
there.
@shiftkey
Copy link
Collaborator

👍 I'll let this sit and see if anyone else has feels on it

@ghuntley
Copy link
Owner

ghuntley commented Dec 1, 2015

👍 for cleaning up but 👎 for merge.

Agree that the page needs some spring cleaning.

Don't agree on this merge because it removes the explanation (however poor it may be) of the following fundamentals:

  • Existence of IEnableLogger
  • Why ReactiveUI ships with it's own logger but explain that usage is not needed (link to configuration w/NLog)
  • Help with debugging observables

Completely agree with the notion of funneling users to the documentation and examples at the Splat repo as it will be more up to date/one less thing to maintain.

@shiftkey
Copy link
Collaborator

shiftkey commented Dec 1, 2015

@ghuntley nice points

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.

3 participants