Bug 1268765 – Review Request: rubygem-grape-entity - A simple facade for managing the relationship between your model and API

Bug 1268765 - Review Request: rubygem-grape-entity - A simple facade for managing the relationship between your model and API

Summary:	Review Request: rubygem-grape-entity - A simple facade for managing the relat...

Status:	NEW

Aliases:	None

Product:	Fedora
Classification:	Fedora
Component:	Package Review (Show other bugs)
Sub Component:
Version:	rawhide
Hardware:	All Linux

Priority	medium Severity medium
Target Milestone:	---
Target Release:	---
Assigned To:	Nobody's working on this, feel free to take it
QA Contact:	Fedora Extras Quality Assurance
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:	1268699 1268701 1268758
Blocks:
	Show dependency tree / graph

Reported:	2015-10-05 04:12 EDT by Ilya Gradina
Modified:	2015-11-02 05:50 EST (History)
CC List:	2 users (show)

See Also:
Fixed In Version:
Doc Type:	Bug Fix
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:
Type:	---
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Groups: None (edit)

Description Ilya Gradina 2015-10-05 04:12:00 EDT

Spec URL: https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-grape-entity.spec
SRPM URL: https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-grape-entity-0.4.8-1.fc24.src.rpm
Description: Extracted from Grape, A Ruby framework for rapid API development with great conventions.
Fedora Account System Username: ilgrad

Comment 1 Ilya Gradina 2015-10-05 04:14:56 EDT

Depends: rubygem(rubocop) = 0.31.0
https://copr.fedoraproject.org/coprs/ilgrad/gitlab/build/123973/

Comment 2 Vít Ondruch 2015-11-02 05:49:56 EST

Could you please drop the Bundler dependency? Something like line:

  sed -i "/require 'bundler'/ s/^/#/" spec/spec_helper.rb

in %check section might be good start. This will very likely avoid also the Rubocop dependency as well as Guard or YARD.

Comment 3 Vít Ondruch 2015-11-02 05:50:39 EST

Neither Pry should be required for build.

Note You need to log in before you can comment on or make changes to this bug.