https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Jamie Nguyen jamieli...@fedoraproject.org changed:
What|Removed |Added
Status|ASSIGNED|CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #18 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Jon Ciesla limburg...@gmail.com changed:
What|Removed |Added
Flags|fedora-cvs? |fedora-cvs+
--
You
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #16 from Jamie Nguyen jamieli...@fedoraproject.org ---
Thanks very much for the review Mukundan! :-)
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Jamie Nguyen jamieli...@fedoraproject.org changed:
What|Removed |Added
Flags||fedora-cvs?
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Bug 1074418 depends on bug 1074412, which changed state.
Bug 1074412 Summary: Review Request: nodejs-reduce-component - An array reduce
component for Node.js
https://bugzilla.redhat.com/show_bug.cgi?id=1074412
What|Removed
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Bug 1074418 depends on bug 1074414, which changed state.
Bug 1074414 Summary: Review Request: nodejs-cookiejar - A simple, persistent
cookie jar system for Node.js
https://bugzilla.redhat.com/show_bug.cgi?id=1074414
What|Removed
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Bug 1074418 depends on bug 1074416, which changed state.
Bug 1074416 Summary: Review Request: nodejs-component-emitter - An event
emitter component for Node.js
https://bugzilla.redhat.com/show_bug.cgi?id=1074416
What|Removed
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #7 from Mukundan Ragavan nonamed...@gmail.com ---
(In reply to Jamie Nguyen from comment #6)
Spec URL: http://jamielinux.fedorapeople.org/supertest/nodejs-superagent.spec
SRPM URL:
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #9 from Jamie Nguyen jamieli...@fedoraproject.org ---
(Fix link)
Spec URL: http://jamielinux.fedorapeople.org/supertest/nodejs-superagent.spec
SRPM URL:
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #8 from Jamie Nguyen jamieli...@fedoraproject.org ---
Oops, sorry! I patched out the use of extend, but forgot to remove BR:
npm(extend).
Spec URL: http://jamielinux.fedorapeople.org/supertest/nodejs-superagent.spec
SRPM URL:
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #10 from Jamie Nguyen jamieli...@fedoraproject.org ---
Spec URL: http://jamielinux.fedorapeople.org/supertest/nodejs-superagent.spec
SRPM URL:
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #11 from Mukundan Ragavan nonamed...@gmail.com ---
Some issues (broken deps?) to be taken care of. Detailed review below -
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #12 from Mukundan Ragavan nonamed...@gmail.com ---
[x]: Changelog in prescribed format.
This is correct. No issues. I looked at the wrong leafpad window!
--
You are receiving this mail because:
You are on the CC list for the
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #13 from Jamie Nguyen jamieli...@fedoraproject.org ---
(In reply to Mukundan Ragavan from comment #11)
Error: Package: nodejs-superagent-0.17.0-4.fc21.noarch
(/nodejs-superagent-0.17.0-4.fc21.noarch)
Requires:
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #14 from Mukundan Ragavan nonamed...@gmail.com ---
I will check the package installation again a later today and approve this
package
Once this is go, I will finish the review of 1074420 also.
--
You are receiving this mail
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Mukundan Ragavan nonamed...@gmail.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Jamie Nguyen jamieli...@fedoraproject.org changed:
What|Removed |Added
Depends On|1074411 |
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Jamie Nguyen jamieli...@fedoraproject.org changed:
What|Removed |Added
Blocks||956806
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #1 from Jamie Nguyen jamieli...@fedoraproject.org ---
All tests pass when using npm installed express@3.5.0.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Jamie Nguyen jamieli...@fedoraproject.org changed:
What|Removed |Added
Blocks||1074420
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Mukundan Ragavan nonamed...@gmail.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
Bug 1074418 depends on bug 1074411, which changed state.
Bug 1074411 Summary: Review Request: nodejs-extend - A port of jQuery.extend
for Node.js
https://bugzilla.redhat.com/show_bug.cgi?id=1074411
What|Removed
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #2 from Mukundan Ragavan nonamed...@gmail.com ---
I guess, I should wait to review this package because of 1074411?
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #3 from Jamie Nguyen jamieli...@fedoraproject.org ---
(In reply to Mukundan Ragavan from comment #2)
I guess, I should wait to review this package because of 1074411?
Hi! Generally a review request should only be tackled once the
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #4 from Mukundan Ragavan nonamed...@gmail.com ---
Sorry - I guess I was not clear. I saw Tom's review (+) for 1074412 and
1074416. I am looking at 1074414 now.
That's why I was asking! :)
--
You are receiving this mail because:
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #5 from Jamie Nguyen jamieli...@fedoraproject.org ---
Oh, sorry, my bad! I hadn't spotted that Tom had already approved them :)
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified
https://bugzilla.redhat.com/show_bug.cgi?id=1074418
--- Comment #6 from Jamie Nguyen jamieli...@fedoraproject.org ---
Spec URL: http://jamielinux.fedorapeople.org/supertest/nodejs-superagent.spec
SRPM URL:
28 matches
Mail list logo