Merge 1.0 alpha branch #46
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[WIP] Reimagining how subscribing observers works
Clean up tests and bump version to 1.0.0-SNAPSHOT
* Move rxgroups-processor to separate project, DRY multi-module dependencies * renamen package to processor, fix spacing * rxgroups-android tests need to depend on processor * Switch line limit back to 100, update gradle to 3.5 * get checkstyle to work with gradle 3.5
… for rx-processor (#38) * Have rxgroups-processor upload build artifact Use variable for repo urls * Make ObseravbleGroup#resubscribe public for generated code. Remove deprecated use of Observable#create * Use getter/setter with JavaDoc * fix checkstyle * Add back javadoc quiet mode * Use subclass to control visability
* Make mulitiple observer support more clear * fix checkystyle * Consistently use the term observableTag for user facing code, and observerTag where necessary internally * Add test for multiple observers, cancelAndRemoveAllWithTag * Update all tests to reflect the use of observerTag instead of null in 1:1 case * remove extra space * Fix javadocs * Remove cancelling by observableTag * remove unused observableTag code * fix another typo
* Add tagged observer to allow for using a non-auto generated observer tag in exceptional circumstances * Support auto resubcription with tagged observer * add tests
* Add AutoTag, safeIntialization, and NonResubscribingTag * revert gradle change * Fix some comments * Change to NonResubscribable * fix test * propogate cancel and remove all change to lifecyclemanager * remove duplicated method * fix test name * Cache missing constructors, add super class initialization support change some access fix some more * Make GroupLifecycleManager properly shadow methods in ObservableGroup * disable check style for mock generated code * reduce chance of concurrent modification exception * Add supression comment filter * Add filecontentsholder to have checkstyle actually read turn off comments * remove duplicated remove code * Add AnnotationProcessor code gen tests * use config build tools version * fix no checkstyle check * add missing new lines * Change version to 1.0.0-alpha1 * Prevent null tag collisions by resorting to NonResubscribingTag, if getTag returns null * add test for custom tagging * eliminate double map lookup
* Migrate to RxJava2 for Observables cleanup * update tests and sample for rxjava2 * update travis * Fix test checkstyle * fix line alignment * Fix some documentation
|
Some other thoughts before 1.0:
|
|
Can you publish it somewhere? It's currently very inconvenient to test. Would be very happy! |
|
@meierjan 1.0.0-alpha2 has been uploaded. Again, this is a pretty rough version of what a full 1.0.0 release would look like, so use with caution. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1.0-alpha has worked internally for a few months now
Currently it only has support for Observable types -- no Completable, Flowable, Single, etc yet, which I think would be necessary for a solid 1.0 release.
Seems a little premature to call it
1.0.0-alphabut seems important to bump the major version as this is very much a breaking change.#34
@felipecsl @elihart