-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Remove builder as a dependency #2445
Remove builder as a dependency #2445
Conversation
Add UPGRADING.md entry
Generated by 🚫 Danger |
UPGRADING.md
Outdated
@@ -3,6 +3,10 @@ Upgrading Grape | |||
|
|||
### Upgrading to >= 2.1.0 | |||
|
|||
#### Builder => XML | |||
|
|||
From now on, if you want to generate XML, you'll need to add `builder` in your Gemfile. |
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.
Let's add some details. Maybe this.
The `builder` gem dependency has been made optional as it's only used when generating XML. If your code does XYZ, add `builder` to `Gemfile`.
See [#2445](https://github.com/ruby-grape/grape/pull/2445) for more information.
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.
The README also needs an update around XML to say builder needs to be added in certain circumstances.
Also, no tests failed?
It's in the our test Gemfile |
Does it make sense to remove it from there and move XML tests to integration? Probably does, can be done in a separate PR. |
Fix #2443