-
Notifications
You must be signed in to change notification settings - Fork 3k
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
fix spelling, add tailing slash #411
fix spelling, add tailing slash #411
Conversation
rxandroid/build.gradle
Outdated
@@ -115,8 +115,8 @@ bintray { | |||
licenses = ['Apache-2.0'] | |||
labels = ['rxjava', 'reactivex'] | |||
websiteUrl = 'https://github.com/ReactiveX/RxAndroid/' | |||
issueTrackerUrl = 'https://github.com/ReactiveX/RxAndroid/issues' | |||
vcsUrl = 'https://github.com/ReactiveX/RxAndroid.git' | |||
issueTrackerUrl = 'https://github.com/ReactiveX/RxAndroid/issues/' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any reason to change this. GitHub doesn't include it in their URLs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought the ~/issue
would be a folder, so I thought it would be redirected to ~/issues/
.
I thought the other url was the same. But I tried it, but it was not.
So, I'll fix and push again. Thanks
rxandroid/build.gradle
Outdated
issueTrackerUrl = 'https://github.com/ReactiveX/RxAndroid/issues' | ||
vcsUrl = 'https://github.com/ReactiveX/RxAndroid.git' | ||
issueTrackerUrl = 'https://github.com/ReactiveX/RxAndroid/issues/' | ||
vcsUrl = 'https://github.com/ReactiveX/RxAndroid.git/' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't see any reason to change this. GitHub doesn't include it in their clone URLs.
rxandroid/build.gradle
Outdated
@@ -84,7 +84,7 @@ install { | |||
// | |||
// Based on NebulaOJOPublishingPlugin (from nebula-bintray-plugin) | |||
artifactory { | |||
contextUrl = 'https://oss.jfrog.org' | |||
contextUrl = 'https://oss.jfrog.org/' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again, what reason is there to add this? Seems like pointless churn.
CHANGES.md
Outdated
@@ -25,9 +25,9 @@ The sections below contain the changes since 2.0.0-RC1. | |||
|
|||
RxAndroid 2.0 has been rewritten from scratch to support RxJava 2.0. | |||
|
|||
The library still offers the same APIs: a scheduler and stream cancelation callback that know about | |||
The library still offers the same APIs: a scheduler and stream cancellation callback that know about |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both of these spellings are considered correct, for what it's worth.
Thanks! |
fix spelling
add trailing slash