Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Final touchups for 0.5.0 release #144

Merged
merged 3 commits into from
Dec 4, 2017
Merged

Final touchups for 0.5.0 release #144

merged 3 commits into from
Dec 4, 2017

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 tyvsmith and jbarr21 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