-
-
Notifications
You must be signed in to change notification settings - Fork 60
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
Bump to v0.4.0 and add UPGRADING docs #14
Conversation
CHANGELOG.md
Outdated
|
||
* Your contribution here. | ||
|
||
### 0.4.0 (2018/07/27) |
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.
This hasn't been released yet, so it shouldn't have a date and say 0.4.0 (Next)
.
|
||
#### All errors that return HTTP codes 400-600 result in a IEX::Errors::ClientError exception | ||
|
||
On previous versions, calling `IEX::Resources::Chart.get` with an invalid option results on a |
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.
Reading this it's pretty clear that we're now doing something different in various API calls. I've opened #15 so that we can handle errors the same way in all methods (ie. raise IEX::Errors::ClientError on every Faraday::ClientError in all calls. Hopefully we don't need to copy paste that code everywhere and implement it in the base class.
Maybe try taking care of #15 first before we merge this? |
15735ed
to
83706c6
Compare
83706c6
to
93e0372
Compare
Perfect. |
close #13
If I understood well, since this release will be a major upgrade
(v0.4.0)
, the next version will actually bev0.4.1
. I noticed on the git history that you @dblock and @jromanovs changed theversion
in two commits, only after indeed releasing at rubygems.org. I already changed the version in advance but added tomorrow's release date.Please let me know if that makes sense.