Bug 1428926

Summary: Review Request: php-justinrainbow-json-schema5 - A library to validate a json schema
Product: [Fedora] Fedora Reporter: Remi Collet <fedora>
Component: Package ReviewAssignee: Randy Barlow <randy>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: package-review, randy
Target Milestone: ---Flags: randy: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-03-31 23:47:50 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
review.txt none

Description Remi Collet 2017-03-03 15:53:56 UTC
Spec URL: https://raw.githubusercontent.com/remicollet/remirepo/master/php/php-justinrainbow-json-schema5/php-justinrainbow-json-schema5.spec
SRPM URL: http://rpms.remirepo.net/SRPMS/php-justinrainbow-json-schema5-5.1.0-1.remi.src.rpm
Description: 
A PHP Implementation for validating JSON Structures against a given Schema.

This package provides the library version %{major}.

See http://json-schema.org/


Fedora Account System Username: remi


-- needed by php-cs-fixer 2.1.1

Comment 1 Randy Barlow 2017-03-21 19:40:34 UTC
Created attachment 1265193 [details]
review.txt

Suggestions, at your option:

* You can drop the rm -rf %{buildroot} in the install section.
  fedora-review put this in the MUST section, but I disagree so I am
  willing to pass as-is.
* You can drop the %defattr statement in the files section.
* You can drop the BuildRoot: line.
* You can delete the %clean section.

Comment 2 Remi Collet 2017-03-22 05:48:50 UTC
Thanks for the review

(In reply to Randy Barlow from comment #1)
> Created attachment 1265193 [details]
> review.txt
> 
> Suggestions, at your option:
> 
> * You can drop the rm -rf %{buildroot} in the install section.
>   fedora-review put this in the MUST section, but I disagree so I am
>   willing to pass as-is.
> * You can drop the %defattr statement in the files section.
> * You can drop the BuildRoot: line.
> * You can delete the %clean section.

Yes, I usually clean those during Fedora import (and will be able to totally clean them in my repo in ~1 week \o/)

Comment 3 Gwyn Ciesla 2017-03-22 12:47:51 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/php-justinrainbow-json-schema5

Comment 4 Fedora Update System 2017-03-22 13:33:23 UTC
php-justinrainbow-json-schema5-5.1.0-1.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-560f805a30

Comment 5 Fedora Update System 2017-03-22 13:33:30 UTC
php-justinrainbow-json-schema5-5.1.0-1.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2017-3b442ce6cb

Comment 6 Fedora Update System 2017-03-22 13:33:35 UTC
php-justinrainbow-json-schema5-5.1.0-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-4534b24bc7

Comment 7 Fedora Update System 2017-03-22 13:33:39 UTC
php-justinrainbow-json-schema5-5.1.0-1.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-6ae1f436ff

Comment 8 Fedora Update System 2017-03-23 14:25:04 UTC
php-justinrainbow-json-schema5-5.2.0-1.fc26 has been pushed to the Fedora 26 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-6ae1f436ff

Comment 9 Fedora Update System 2017-03-23 18:48:40 UTC
php-justinrainbow-json-schema5-5.2.0-1.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-4534b24bc7

Comment 10 Fedora Update System 2017-03-23 19:22:50 UTC
php-justinrainbow-json-schema5-5.2.0-1.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-3b442ce6cb

Comment 11 Fedora Update System 2017-03-23 19:23:42 UTC
php-justinrainbow-json-schema5-5.2.0-1.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-560f805a30

Comment 12 Fedora Update System 2017-03-31 23:47:50 UTC
php-justinrainbow-json-schema5-5.2.0-1.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.

Comment 13 Fedora Update System 2017-04-01 00:24:39 UTC
php-justinrainbow-json-schema5-5.2.0-1.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.

Comment 14 Fedora Update System 2017-04-01 17:26:55 UTC
php-justinrainbow-json-schema5-5.2.0-1.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report.

Comment 15 Fedora Update System 2017-04-07 13:49:06 UTC
php-justinrainbow-json-schema5-5.2.0-1.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report.