Closed drn closed 7 years ago
2 Warnings | |
---|---|
:warning: | There’re library changes, but not tests. That’s OK as long as you’re refactoring existing code. |
:warning: | Unless you’re refactoring existing code, please update CHANGELOG.md. |
Here's an example of a CHANGELOG.md entry:
* [#71](https://github.com/ruby-grape/grape-active_model_serializers/pull/71): Ensure support for sequel models and datasets - [@drn](https://github.com/drn).
Generated by :no_entry_sign: danger
closed this in favor of https://github.com/ruby-grape/grape-active_model_serializers/pull/74
Addresses https://github.com/ruby-grape/grape-active_model_serializers/issues/70. Need to add specs that test both Sequel / AR support before this is ready to be merged