Bug 1157700 - Review Request: php-ocramius-code-generator-utils - A set of code generator utilities built on top of PHP-Parsers
Summary: Review Request: php-ocramius-code-generator-utils - A set of code generator u...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Remi Collet
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1157708
TreeView+ depends on / blocked
 
Reported: 2014-10-27 14:16 UTC by Shawn Iwinski
Modified: 2014-11-24 21:22 UTC (History)
2 users (show)

Fixed In Version: php-ocramius-code-generator-utils-0.3.0-2.el7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-11-16 14:47:23 UTC
Type: ---
Embargoed:
fedora: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)
phpci.log (7.14 KB, text/plain)
2014-11-04 08:52 UTC, Remi Collet
no flags Details
review.txt (7.10 KB, text/plain)
2014-11-04 08:53 UTC, Remi Collet
no flags Details

Description Shawn Iwinski 2014-10-27 14:16:28 UTC
Spec URL: https://raw.githubusercontent.com/siwinski/rpms/375ca5028ea7c8c7df11df954265a85a417ffee3/php-ocramius-code-generator-utils.spec

SRPM URL: https://siwinski.fedorapeople.org/copr/php-experimental/php-ocramius-code-generator-utils-0.3.0-1.fc20.src.rpm

Description:
A set of code generator utilities built on top of PHP-Parsers that ease its use
when combined with Reflection.


Fedora Account System Username: siwinski


COPR build: http://copr.fedoraproject.org/coprs/siwinski/php-experimental/build/55152/

Comment 1 Remi Collet 2014-11-04 08:52:26 UTC
Created attachment 953491 [details]
phpci.log

 phpCompatInfo version 3.5.0 static analyze run in src directory

Comment 2 Remi Collet 2014-11-04 08:53:06 UTC
Created attachment 953492 [details]
review.txt

Generated by fedora-review 0.5.2 (63c24cb) last change: 2014-07-14

Comment 3 Remi Collet 2014-11-04 08:55:01 UTC
Minor cleanup (no blocker)

- ensure you use the correct spec (see review.txt) without the debug "find"
- consider to silent the include (in test autoloader) to avoid some warning at build time


Everything seems OK.

=== APPROVED ===

Comment 4 Shawn Iwinski 2014-11-04 17:45:09 UTC
(In reply to Remi Collet from comment #3)
> Minor cleanup (no blocker)
> 
> - ensure you use the correct spec (see review.txt) without the debug "find"

Oooops, my bad.  That was a debug b/c of an issue w/ the previous PHPParser release.

> - consider to silent the include (in test autoloader) to avoid some warning
> at build time

I'm not sure how that un-silenced.  Perhaps during my debug.

> Everything seems OK.
> 
> === APPROVED ===

I'll make sure everything is cleaned up after initial import.

Comment 5 Shawn Iwinski 2014-11-04 17:49:31 UTC
THANKS for the review!


New Package SCM Request
=======================
Package Name: php-ocramius-code-generator-utils
Short Description: A set of code generator utilities built on top of PHP-Parsers
Upstream URL: https://github.com/Ocramius/CodeGenerationUtils
Owners: siwinski
Branches: f21 epel7
InitialCC:

Comment 6 Gwyn Ciesla 2014-11-05 12:09:24 UTC
Git done (by process-git-requests).

Comment 7 Fedora Update System 2014-11-06 16:38:28 UTC
php-ocramius-code-generator-utils-0.3.0-2.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/php-ocramius-code-generator-utils-0.3.0-2.fc21

Comment 8 Fedora Update System 2014-11-06 16:38:34 UTC
php-ocramius-code-generator-utils-0.3.0-2.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/php-ocramius-code-generator-utils-0.3.0-2.el7

Comment 9 Fedora Update System 2014-11-07 05:32:53 UTC
php-ocramius-code-generator-utils-0.3.0-2.fc21 has been pushed to the Fedora 21 testing repository.

Comment 10 Fedora Update System 2014-11-16 14:47:23 UTC
php-ocramius-code-generator-utils-0.3.0-2.fc21 has been pushed to the Fedora 21 stable repository.

Comment 11 Fedora Update System 2014-11-24 21:22:58 UTC
php-ocramius-code-generator-utils-0.3.0-2.el7 has been pushed to the Fedora EPEL 7 stable repository.


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