chore(Observable.ajax): change method to optional on interface, since…

[elliotschi]

Description:

Related issue (if exists):

… it gets set to GET by default

[coveralls]

Coverage remained the same at 97.113% when pulling 01dfd7e on elliotschi:master into 8b60e8d on ReactiveX:master.

[kwonoj]

Change itself makes sense, only thing bit bothers me is since AjaxRequest is now completely optional interfaces, so AjaxObservable.create({}) is valid construction. Seems not an issue, just looking bit odd.

[kwonoj]

I wouldn't worry that much about AjaxObservable.create({}), change itself LGTM. Just will leave this opened bit more to see if there's any concerns around.

[elliotschi]

@kwonoj sounds good

[coveralls]

Coverage remained the same at 97.115% when pulling 09fb79e on elliotschi:master into 80b817f on ReactiveX:master.

[coveralls]

Coverage remained the same at 97.115% when pulling 09fb79e on elliotschi:master into 80b817f on ReactiveX:master.

[coveralls]

Coverage remained the same at 97.115% when pulling 35a5d43 on elliotschi:master into 7b23e88 on ReactiveX:master.

[coveralls]

Coverage remained the same at 97.115% when pulling 0103940 on elliotschi:master into cb40e92 on ReactiveX:master.

[jayphelps]

LGTM, please rebase.

[coveralls]

Coverage remained the same at 97.115% when pulling 35a5d43 on elliotschi:master into 7b23e88 on ReactiveX:master.

[kwonoj]

@jayphelps you're so fast, I was about to click merge btn and it was already merged! 💨

[jayphelps]

👊 fastest merge in the West.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.