Difference between revisions of "Review Queue"

From D Wiki
Jump to: navigation, search
(Current Review Queue)
(Moved color to review queue on advice from jpf91)
 
(89 intermediate revisions by 17 users not shown)
Line 4: Line 4:
  
 
== Current Review Queue ==
 
== Current Review Queue ==
 +
 +
This list only contains only additions that are currently under review and have been publicly announced for voting on the newsgroup.
 +
It should be updated and removed by the review manager after the voting period ended.
  
 
{| class="wikitable sortable" border="1"
 
{| class="wikitable sortable" border="1"
Line 9: Line 12:
 
! scope="col" | Package
 
! scope="col" | Package
 
! scope="col" | Champion
 
! scope="col" | Champion
 +
! scope="col" | Review Manager
 
! scope="col" class="unsortable" | Code
 
! scope="col" class="unsortable" | Code
 
! scope="col" class="unsortable" | Documentation
 
! scope="col" class="unsortable" | Documentation
Line 14: Line 18:
 
! scope="col" | Status
 
! scope="col" | Status
 
|-
 
|-
| std.serialize
+
| std.experimental.color
 +
  || Manu Evans
 +
  ||
 +
  || [https://github.com/dlang/phobos/pull/2845 Code/Pull Request]
 +
  || [https://code.dlang.org/packages/color color on DUB]
 +
  || [https://forum.dlang.org/post/mailman.41.1472618624.2965.digitalmars-d@puremagic.com Forum thread]
 +
  || <font color=black>Needs review manager, users, testers</font>
 +
|}
 +
 
 +
== Work in progress ==
 +
 
 +
Theses additions are still in work or currently being reworked due to feedback.
 +
If you are interested, feel free to help out.
 +
 
 +
{| class="wikitable sortable" border="1"
 +
|-
 +
! scope="col" | Package
 +
! scope="col" | Champion
 +
! scope="col" | Review Manager
 +
! scope="col" class="unsortable" | Code
 +
! scope="col" class="unsortable" | Documentation
 +
! scope="col" class="unsortable" | Other
 +
! scope="col" | Status
 +
|-
 +
| [[Review/std.lexer|std.lexer]]
 +
  || Brian Schott
 +
  ||
 +
  || [https://github.com/Hackerpilot/Dscanner/blob/master/std/lexer.d Code]
 +
  || [http://hackerpilot.github.io/experimental/std_lexer/phobos/lexer.html Docs]
 +
  || [http://forum.dlang.org/post/aiipgaqyddjijpjiulfu@forum.dlang.org Review Thread]
 +
  || <font color=red>[https://github.com/Hackerpilot/Dscanner/issues/316 Needs work]</font>
 +
|-
 +
| std.decimal
 +
  || Paul D. Anderson
 +
  ||
 +
  || [https://github.com/andersonpd/eris Code]
 +
  || [https://github.com/andersonpd/eris/wiki Docs]
 +
  || [http://www.speleotrove.com/decimal/decarith.pdf Specification]
 +
  || <font color=red>[https://github.com/andersonpd/eris/issues/6 Needs work]</font>
 +
|-
 +
| [[Review/std.serialization|std.serialization]]
 
   || Jacob Carlborg
 
   || Jacob Carlborg
 +
  ||
 
   || [https://github.com/jacob-carlborg/phobos/tree/serialization Code]
 
   || [https://github.com/jacob-carlborg/phobos/tree/serialization Code]
 
   || [https://dl.dropboxusercontent.com/u/18386187/docs/std.serialization/index.html  Docs]  
 
   || [https://dl.dropboxusercontent.com/u/18386187/docs/std.serialization/index.html  Docs]  
Line 22: Line 67:
  
 
[http://forum.dlang.org/post/hsnmxykmoytfvwroikzk@forum.dlang.org Review thread 2]
 
[http://forum.dlang.org/post/hsnmxykmoytfvwroikzk@forum.dlang.org Review thread 2]
   || <font color=darkorange>Incorporating review comments</font>
+
   || <font color=red>[https://github.com/jacob-carlborg/phobos/issues/3 Needs work]</font>
 +
|-
 +
| std.io
 +
  || Steven Schveighoffer
 +
  ||
 +
  || [https://github.com/schveiguy/iopipe Code]
 +
  ||
 +
  || [http://forum.dlang.org/post/na14ul$30uo$1@digitalmars.com RFC Thread]
 +
  || <font color=black>[https://github.com/schveiguy/iopipe/issues/1 Work in progress]</font>
 
|-
 
|-
| std.d.lexer
+
| std.experimental.si
   || Brian Schott
+
  || David Nadlinger, Per Nordlöw
   || [https://github.com/Hackerpilot/Dscanner/blob/range-based-lexer/std/d/lexer.d Code]
+
   ||
  || [http://hackerpilot.github.io/experimental/std_lexer/phobos/lexer.html Docs]
+
   || [https://github.com/nordlow/units-d Code]
 
   ||
 
   ||
   || <font color=darkorange>Ready for review</font>
+
  || [http://forum.dlang.org/post/io1vgo$1fnc$1@digitalmars.com RFC Thread]
 +
   || <font color=black>Work in progress</font>
 
|-
 
|-
| std.compression.lz77
+
| std.experimental.safeint
   || Walter Bright
+
  || Robert burner Schadek
   || [https://github.com/D-Programming-Language/phobos/pull/1335 Code]
+
   ||  
 +
   || [https://github.com/D-Programming-Language/phobos/pull/3389 Code/Pull Request]
 +
  || [http://burner.github.io/phobos/phobos-prerelease/std_experimental_safeint.html Docs]
 +
  || [http://forum.dlang.org/post/ulaywuujlhhubxiidagw@forum.dlang.org Forum thread]
 +
  || <font color=black>superseded by checkedint</font>
 +
|-
 +
| std.experimental.checkedint
 +
  || Thomas Stuart Bockman aka. tsbockmann  ||
 +
  || [https://github.com/dlang/phobos/pull/4407 Code/Pull Request]
 +
  || [Docs]
 +
  || [http://forum.dlang.org/post/jxaisipbdqfifpncnvaa@forum.dlang.org Forum thread]
 +
  || <font color=black></font>
 +
|-
 +
| std.experimental.perpetual
 +
  || Sergei Degtiarev
 
   ||  
 
   ||  
   || [http://forum.dlang.org/post/koqrq8$h4b$1@digitalmars.com Discussion]
+
   || [https://github.com/D-Programming-Language/phobos/pull/3625 Code/Pull Request]
   || <font color=darkorange>Ready for review</font>
+
  || [https://sdegtiarev.github.io/ Docs]
 +
  || [https://forum.dlang.org/post/stesmdubyyldrseepqqz@forum.dlang.org Forum thread]
 +
   || <font color=black>Needs users/testers</font>
 +
|}
 +
 
 +
== Stalled Reviews ==
 +
 
 +
These additions got stalled or abandoned.
 +
 
 +
{| class="wikitable sortable" border="1"
 +
|-
 +
! scope="col" | Package
 +
! scope="col" | Champion
 +
! scope="col" | Review Manager
 +
! scope="col" class="unsortable" | Code
 +
! scope="col" class="unsortable" | Documentation
 +
! scope="col" class="unsortable" | Other
 +
! scope="col" | Status
 
|-
 
|-
| std.idioms
+
| std.benchmark
   || Idan Arye
+
  || Andrei Alexandrecu
   || [https://github.com/D-Programming-Language/phobos/pull/1294 Code]
+
   ||
   || [http://someboddy.github.io/phobos/ddocs/for-idioms/idioms.html Docs]
+
   || [https://github.com/D-Programming-Language/phobos/pull/529 Code]
   || [http://forum.dlang.org/post/fofbrlqruxbevnxchxdp@forum.dlang.org Discussion Thread]
+
   || [http://erdani.com/d/web/phobos-prerelease/std_benchmark.html Docs]  
   || <font color=darkorange>Ready for review</font>
+
   ||  
 +
   || <font color=black>Abandoned</font>
 
|-
 
|-
| std.decimal
+
| dtoh
   || Paul D. Anderson
+
   || Adam D. Ruppe
   || [https://github.com/andersonpd/decimal Code]
+
  ||
   || [https://github.com/andersonpd/decimal/wiki Docs]  
+
   || [https://github.com/adamdruppe/tools/blob/dtoh/dtoh.d Code]
   || [http://www.speleotrove.com/decimal/decarith.pdf Specification]
+
   || [http://forum.dlang.org/post/lgspgg$2i8l$1@digitalmars.com RFC Thread]
   || <font color=darkorange>Ready for comments</font>
+
   || [https://github.com/D-Programming-Language/tools/pull/39 Pull request]
 +
   || <font color=black>On hold</font>
 
|-
 
|-
 
| std.xml2
 
| std.xml2
 
   || Michael Rynn
 
   || Michael Rynn
 +
  ||
 
   || [https://launchpad.net/d2-xml Code]
 
   || [https://launchpad.net/d2-xml Code]
 
   || [http://dsource.org/projects/xmlp Docs]  
 
   || [http://dsource.org/projects/xmlp Docs]  
 
   ||
 
   ||
   || <font color=darkorange>Ready for comments</font>
+
   || <font color=black>Abandoned</font>
 
|-
 
|-
| std.signal
+
| std.halffloat
   || Robert Klotzner
+
  || Walter Bright
   || [https://github.com/phobos-x/phobosx/blob/master/source/phobosx/signal.d Code]
+
   ||
   || [https://vhios.dyndns.org/signal.html Docs]
+
   || [https://github.com/D-Programming-Language/phobos/pull/2207 Code]
  || Replacement for std.signals
+
  ||
   || <font color=darkorange>Ready for review</font>
+
  || [https://github.com/D-Programming-Language/phobos/pull/2207 Pull request]
 +
  || <font color=black>On hold</font>
 +
|-
 +
| std.compression.lz77
 +
  || Walter Bright
 +
  ||
 +
  || [https://github.com/D-Programming-Language/phobos/pull/1335 Code]
 +
  ||
 +
   || [http://forum.dlang.org/post/koqrq8$h4b$1@digitalmars.com Discussion]
 +
   || <font color=black>Abandoned</font>
 
|-
 
|-
 
| std.log
 
| std.log
 
   || Jose Armando Garcia
 
   || Jose Armando Garcia
 +
  ||
 
   || [https://github.com/D-Programming-Language/phobos/pull/432/files Code]
 
   || [https://github.com/D-Programming-Language/phobos/pull/432/files Code]
 
   || [http://jsancio.github.com/phobos/phobos/std_log.html Docs]  
 
   || [http://jsancio.github.com/phobos/phobos/std_log.html Docs]  
 
   || [http://forum.dlang.org/post/jhbbfd$1tmk$1@digitalmars.com Review thread]
 
   || [http://forum.dlang.org/post/jhbbfd$1tmk$1@digitalmars.com Review thread]
   || <font color=black>On hold</font>
+
   || <font color=black>Abandoned</font>
|-
 
| std.logger
 
  || Robert Schadek
 
  || [https://github.com/D-Programming-Language/phobos/pull/1500/files Code]
 
  || [http://burner.github.io/phobos/phobos-prerelease/std_logger.html Docs]
 
  || [http://forum.dlang.org/post/mailman.313.1377180809.1719.digitalmars-d@puremagic.com Discussion thread]
 
  || <font color=black>Work in progress</font>
 
|-
 
| std.benchmark
 
  || Andrei Alexandrecu
 
  || [https://github.com/D-Programming-Language/phobos/pull/529 Code]
 
  || [http://erdani.com/d/web/phobos-prerelease/std_benchmark.html Docs]
 
  ||
 
  || <font color=black>Work in progress</font>
 
|-
 
| std.io
 
  || Steven Schveighoffer
 
  || [https://github.com/schveiguy/phobos/blob/new-io/std/io.d Code]
 
  ||
 
  ||
 
  || <font color=black>Work in progress</font>
 
 
|}
 
|}
  
Line 108: Line 185:
 
! scope="col" | Date included in Phobos
 
! scope="col" | Date included in Phobos
 
! scope="col" | Commit
 
! scope="col" | Commit
 +
|-
 +
| 2016-01-02
 +
  || [[Review/std.experimental.ndslice|std.experimental.ndslice]]
 +
  || Multidimensional ranges and arrays
 +
  || Ilya Yaroshenko
 +
  |bgcolor=limegreen| Yes
 +
  || 12+1/0
 +
  || Jack Stouffer
 +
  || 2016-01-02
 +
  || [https://github.com/9il/phobos/commit/54a6d72bb8875cfc5d0d6df6fcf20242c95b9749 54a6d72]
 +
|-
 +
| 2015-10-08
 +
  || [https://github.com/D-Programming-Language/phobos/pull/3207 std.experimental.testing]
 +
  || Named unittests
 +
  || Atila Neves
 +
  |bgcolor=gold| No / Postponed
 +
  || 1+1/3
 +
  || Robert burner Schadek
 +
  ||
 +
  ||
 +
|-
 +
| 2015-09-17
 +
| [https://github.com/D-Programming-Language/phobos/pull/3389 std.experimental.safeint]
 +
  || Checked integers with dedicated NaN value
 +
  || Robert burner Schadek
 +
  |bgcolor=gold| No / Postponed
 +
  || 1/9
 +
  || -
 +
  ||
 +
  ||
 +
|-
 +
| 2015-07-08
 +
  || [[Review/std.experimental.allocator|std.experimental.allocator]]
 +
  || Allocator package
 +
  || Andrei Alexandrescu
 +
  |bgcolor=limegreen| Yes
 +
  || 25/0
 +
  || Dicebot (Михаил Страшун)
 +
  || TBD
 +
  || TBD
 +
|-
 +
| 2015-01-26
 +
  || [[Review/std.logger|std.logger]]
 +
  || Standard logging package
 +
  || Robert Schadek
 +
  |bgcolor=limegreen| Yes
 +
  ||
 +
  || Dicebot (Михаил Страшун)
 +
  || 2015-01-26
 +
  || [https://github.com/D-Programming-Language/phobos/commit/9d54553410d66881a480b1186d7d625dfe1b86c1 9d54553]
 +
|-
 +
| 2014-01-20
 +
  || [[Review/std.logger|std.logger]]
 +
  || Standard logging package
 +
  || Robert Schadek
 +
  |bgcolor=gold| No / Postponed
 +
  || 6/6
 +
  || Dicebot (Михаил Страшун)
 +
  ||
 +
  ||
 +
|-
 +
| 2014-04-08
 +
  || dtoh
 +
  || A tool for creating C/C++ bindings for D modules
 +
  || Adam D. Ruppe
 +
  |bgcolor=gold| Suspended
 +
  || -
 +
  || Jacob Carlborg
 +
  || -
 +
  || -
 +
|-
 +
| 2014-01-20
 +
  || [[Review/std.signal|std.signal]]
 +
  || Replacement for Phobos signal module
 +
  || Robert Klotzner
 +
  |bgcolor=gold| No / Postponed
 +
  || 3/0/3
 +
  || Dicebot (Михаил Страшун)
 +
  ||
 +
  ||
 +
|-
 +
| 2013-10-13
 +
  || [[Review/std.d.lexer|std.d.lexer]]
 +
  || Lexing of D code
 +
  || Brian Schott
 +
  |bgcolor=gold| No / Postponed
 +
  || 11/3/7
 +
  || Dicebot (Михаил Страшун)
 +
  ||
 +
  ||
 
|-
 
|-
 
| 2013-05-27
 
| 2013-05-27
Line 117: Line 284:
 
   || Jesse Phillips
 
   || Jesse Phillips
 
   || 2013-08-08
 
   || 2013-08-08
   || http://goo.gl/rD9I1W
+
   || [https://github.com/D-Programming-Language/phobos/commit/c109594456fb929447741ade09928b3cc679069e c1095944]
 
|-
 
|-
 
| 2013-04-20
 
| 2013-04-20
Line 127: Line 294:
 
   || Jesse Phillips
 
   || Jesse Phillips
 
   || 2013-05-07
 
   || 2013-05-07
   || http://goo.gl/LJhQy
+
   || [https://github.com/D-Programming-Language/phobos/commit/9d987513855ce00440b6f6af255e2a95ab8eab9a 9d987513]
 
|-
 
|-
 
| 2012-08-29
 
| 2012-08-29
Line 137: Line 304:
 
   || Dmitry Olshansky
 
   || Dmitry Olshansky
 
   || 2012-09-16
 
   || 2012-09-16
   || http://goo.gl/ibqpA
+
   || [https://github.com/D-Programming-Language/phobos/commit/688e1d40573775e74b7b03d259e9c194746255b8 688e1d40]
 
|-
 
|-
 
| 2012-06-27
 
| 2012-06-27
Line 147: Line 314:
 
   || Dmitry Olshansky
 
   || Dmitry Olshansky
 
   || 2012-07-02
 
   || 2012-07-02
   || http://goo.gl/QNmRy
+
   || [https://github.com/D-Programming-Language/phobos/commit/8ac3aea10b1ea1f78506ca9d73285032c9f12499 8ac3aea1]
 
|-
 
|-
 
| 2012-03-12
 
| 2012-03-12
Line 167: Line 334:
 
   || DSimcha
 
   || DSimcha
 
   || 2012-01-28
 
   || 2012-01-28
   || http://goo.gl/l7Yyk
+
   || [https://github.com/D-Programming-Language/phobos/commit/ddff40918b1e69170de6d070d68b82ab854e21c0 ddff4091]
 
|-
 
|-
 
| 2011-11-20
 
| 2011-11-20
Line 187: Line 354:
 
   || Jesse Phillips
 
   || Jesse Phillips
 
   || 2011-11-06
 
   || 2011-11-06
   || http://goo.gl/PcoHr
+
   || [https://github.com/D-Programming-Language/phobos/commit/a627abd3916d9aeb5e1899bd93fa539a16109f49 a627abd3]
 
|-
 
|-
 
| 2011-09-01
 
| 2011-09-01

Latest revision as of 12:30, 9 September 2016

This is the review queue of pending packages to get into the D Standard Library (Phobos). The review process is explained here.

If you want a project to put on this list, please look at the wish list.

Current Review Queue

This list only contains only additions that are currently under review and have been publicly announced for voting on the newsgroup. It should be updated and removed by the review manager after the voting period ended.

Package Champion Review Manager Code Documentation Other Status
std.experimental.color Manu Evans Code/Pull Request color on DUB Forum thread Needs review manager, users, testers

Work in progress

Theses additions are still in work or currently being reworked due to feedback. If you are interested, feel free to help out.

Package Champion Review Manager Code Documentation Other Status
std.lexer Brian Schott Code Docs Review Thread Needs work
std.decimal Paul D. Anderson Code Docs Specification Needs work
std.serialization Jacob Carlborg Code Docs Examples

Review thread 1

Review thread 2

Needs work
std.io Steven Schveighoffer Code RFC Thread Work in progress
std.experimental.si David Nadlinger, Per Nordlöw Code RFC Thread Work in progress
std.experimental.safeint Robert burner Schadek Code/Pull Request Docs Forum thread superseded by checkedint
std.experimental.checkedint Thomas Stuart Bockman aka. tsbockmann Code/Pull Request [Docs] Forum thread
std.experimental.perpetual Sergei Degtiarev Code/Pull Request Docs Forum thread Needs users/testers

Stalled Reviews

These additions got stalled or abandoned.

Package Champion Review Manager Code Documentation Other Status
std.benchmark Andrei Alexandrecu Code Docs Abandoned
dtoh Adam D. Ruppe Code RFC Thread Pull request On hold
std.xml2 Michael Rynn Code Docs Abandoned
std.halffloat Walter Bright Code Pull request On hold
std.compression.lz77 Walter Bright Code Discussion Abandoned
std.log Jose Armando Garcia Code Docs Review thread Abandoned

Previously Reviewed

Date Module Description Author Accepted Votes (yes/no) Review Manager Date included in Phobos Commit
2016-01-02 std.experimental.ndslice Multidimensional ranges and arrays Ilya Yaroshenko Yes 12+1/0 Jack Stouffer 2016-01-02 54a6d72
2015-10-08 std.experimental.testing Named unittests Atila Neves No / Postponed 1+1/3 Robert burner Schadek
2015-09-17 std.experimental.safeint Checked integers with dedicated NaN value Robert burner Schadek No / Postponed 1/9 -
2015-07-08 std.experimental.allocator Allocator package Andrei Alexandrescu Yes 25/0 Dicebot (Михаил Страшун) TBD TBD
2015-01-26 std.logger Standard logging package Robert Schadek Yes Dicebot (Михаил Страшун) 2015-01-26 9d54553
2014-01-20 std.logger Standard logging package Robert Schadek No / Postponed 6/6 Dicebot (Михаил Страшун)
2014-04-08 dtoh A tool for creating C/C++ bindings for D modules Adam D. Ruppe Suspended - Jacob Carlborg - -
2014-01-20 std.signal Replacement for Phobos signal module Robert Klotzner No / Postponed 3/0/3 Dicebot (Михаил Страшун)
2013-10-13 std.d.lexer Lexing of D code Brian Schott No / Postponed 11/3/7 Dicebot (Михаил Страшун)
2013-05-27 std.uni More complete API Dmitry Olshansky Yes 15/0 Jesse Phillips 2013-08-08 c1095944
2013-04-20 std.process Revamped API Lars T. Kyllingstad / Steven Schveighoffer Yes 18/0 Jesse Phillips 2013-05-07 9d987513
2012-08-29 std.digest Digests (SHA; CRC) Johannes Pfau Yes 6/1 Dmitry Olshansky 2012-09-16 688e1d40
2012-06-27 std.uuid UUIDs Johannes Pfau Yes 10/0 Dmitry Olshansky 2012-07-02 8ac3aea1
2012-03-12 std.log Logging José Armando García Suspended - David Nadlinger - -
2011-12-29 std.net.curl CURL wrapper Jonas Drewsen Yes 14/0 DSimcha 2012-01-28 ddff4091
2011-11-20 std.csv csv handling Jesse Phillips Yes 5/1 DSimcha ? ?
2011-10-08 std.regex Regular Expressions Dmitry Olshansky Yes 10/0 Jesse Phillips 2011-11-06 a627abd3
2011-09-01 std.net.curl CURL wrapper Jonas Drewsen Postponed - David Nadlinger - -
2011-08-12 std.path Path manipulation Lars Kyllingstad Yes 22/0 DSimcha ? ?