Skip to content

Final touchups for 0.5.0 release#144

Merged
ZacSweers merged 3 commits intomasterfrom
z/readmeUpdates
Dec 4, 2017
Merged

Final touchups for 0.5.0 release#144
ZacSweers merged 3 commits intomasterfrom
z/readmeUpdates

Conversation

@ZacSweers
Copy link
Collaborator

This is a few housecleaning changes:

  • README updates
  • Couple gradle fixes
  • Promote delegateObserver() APIs to stable since we'll keep these. This ensures they're compatible with subscribeWith in the future, and we can keep the library flexible.

Puts the buildscript/repo config back in the top level (I think they were accidentally moved)

Switches to file base api for checkstyle destination, previous was deprecated
This gives the README some much-needed love.
* Update for the new as() API
* Consolidate duplicate plugin sections (wat)
* Document public AutoDisposing observers and their delegateObserver() methods
* Some various other fine tunings
This is not going to be changed in the future, as it's useful for subscribeWith scenarios too and we don't need to unnecessarily tie it down
@ZacSweers ZacSweers requested review from jbarr21 and tyvsmith December 4, 2017 03:44
@ZacSweers ZacSweers self-assigned this Dec 4, 2017
@ZacSweers ZacSweers added this to the 0.5.0 milestone Dec 4, 2017
@ZacSweers ZacSweers mentioned this pull request Dec 4, 2017
@ZacSweers
Copy link
Collaborator Author

Did an over the shoulder review, merging since it's not functional

@ZacSweers ZacSweers merged commit 87330fb into master Dec 4, 2017
@ZacSweers ZacSweers deleted the z/readmeUpdates branch December 4, 2017 03:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant