Difference between revisions of "Review Queue"

From D Wiki
Jump to: navigation, search
m (Current Review Queue)
(Moved color to review queue on advice from jpf91)
 
(45 intermediate revisions by 11 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
 
|-
 
|-
| [[Review/std.logger|std.logger]]
+
| std.experimental.color
   || Robert Schadek
+
  || Manu Evans
   || [https://github.com/D-Programming-Language/phobos/pull/1500/files Code]
+
   ||  
   || [http://burner.github.io/phobos/phobos-prerelease/std_logger.html Docs]  
+
   || [https://github.com/dlang/phobos/pull/2845 Code/Pull Request]
   || [http://forum.dlang.org/post/mailman.313.1377180809.1719.digitalmars-d@puremagic.com Discussion thread]
+
   || [https://code.dlang.org/packages/color color on DUB]
[http://forum.dlang.org/post/wuahsrjdcmdifnfgjfgs@forum.dlang.org Review thread 1]
+
   || [https://forum.dlang.org/post/mailman.41.1472618624.2965.digitalmars-d@puremagic.com Forum thread]
   || <font color=red>Review in progress</font>
+
   || <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]]
 
| [[Review/std.lexer|std.lexer]]
 
   || Brian Schott
 
   || Brian Schott
 +
  ||
 
   || [https://github.com/Hackerpilot/Dscanner/blob/master/std/lexer.d Code]
 
   || [https://github.com/Hackerpilot/Dscanner/blob/master/std/lexer.d Code]
 
   || [http://hackerpilot.github.io/experimental/std_lexer/phobos/lexer.html Docs]
 
   || [http://hackerpilot.github.io/experimental/std_lexer/phobos/lexer.html Docs]
 
   || [http://forum.dlang.org/post/aiipgaqyddjijpjiulfu@forum.dlang.org Review Thread]
 
   || [http://forum.dlang.org/post/aiipgaqyddjijpjiulfu@forum.dlang.org Review Thread]
   || <font color=red>Ready for voting</font>
+
   || <font color=red>[https://github.com/Hackerpilot/Dscanner/issues/316 Needs work]</font>
|-
 
| std.idioms
 
  || Idan Arye
 
  || [https://github.com/D-Programming-Language/phobos/pull/1294 Code]
 
  || [http://someboddy.github.io/phobos/ddocs/for-idioms/idioms.html Docs]
 
  || [http://forum.dlang.org/post/fofbrlqruxbevnxchxdp@forum.dlang.org Discussion Thread]
 
  || <font color=darkorange>Ready for review</font>
 
 
|-
 
|-
 
| std.decimal
 
| std.decimal
 
   || Paul D. Anderson
 
   || Paul D. Anderson
 +
  ||
 
   || [https://github.com/andersonpd/eris Code]
 
   || [https://github.com/andersonpd/eris Code]
 
   || [https://github.com/andersonpd/eris/wiki Docs]  
 
   || [https://github.com/andersonpd/eris/wiki Docs]  
 
   || [http://www.speleotrove.com/decimal/decarith.pdf Specification]
 
   || [http://www.speleotrove.com/decimal/decarith.pdf Specification]
   || <font color=darkorange>Ready for comments</font>
+
   || <font color=red>[https://github.com/andersonpd/eris/issues/6 Needs work]</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>
 
 
|-
 
|-
 
| [[Review/std.serialization|std.serialization]]
 
| [[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 58: 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=black>Work in progress</font>
+
   || <font color=red>[https://github.com/jacob-carlborg/phobos/issues/3 Needs work]</font>
 
|-
 
|-
 
| std.io
 
| std.io
 
   || Steven Schveighoffer
 
   || Steven Schveighoffer
   || [https://github.com/schveiguy/phobos/blob/new-io/std/io.d Code]
+
  ||
 +
   || [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.experimental.si
 +
  || David Nadlinger, Per Nordlöw
 +
  ||
 +
  || [https://github.com/nordlow/units-d Code]
 +
  ||
 +
  || [http://forum.dlang.org/post/io1vgo$1fnc$1@digitalmars.com RFC Thread]
 +
  || <font color=black>Work in progress</font>
 +
|-
 +
| std.experimental.safeint
 +
  || Robert burner Schadek
 
   ||  
 
   ||  
   || <font color=black>Work in progress</font>
+
  || [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.xml2
+
| std.experimental.perpetual
   || Michael Rynn
+
  || Sergei Degtiarev
   || [https://launchpad.net/d2-xml Code]
+
   ||  
   || [http://dsource.org/projects/xmlp Docs]  
+
   || [https://github.com/D-Programming-Language/phobos/pull/3625 Code/Pull Request]
   ||
+
   || [https://sdegtiarev.github.io/ Docs]
   || <font color=black>On hold</font> / <font color=black>Abandoned</font>
+
   || [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"
 
|-
 
|-
| std.log
+
! scope="col" | Package
  || Jose Armando Garcia
+
! scope="col" | Champion
  || [https://github.com/D-Programming-Language/phobos/pull/432/files Code]
+
! scope="col" | Review Manager
  || [http://jsancio.github.com/phobos/phobos/std_log.html Docs]
+
! scope="col" class="unsortable" | Code
  || [http://forum.dlang.org/post/jhbbfd$1tmk$1@digitalmars.com Review thread]
+
! scope="col" class="unsortable" | Documentation
  || <font color=black>On hold</font> / <font color=black>Abandoned</font>
+
! scope="col" class="unsortable" | Other
 +
! scope="col" | Status
 
|-
 
|-
| std.compression.lz77
+
| std.benchmark
   || Walter Bright
+
  || Andrei Alexandrecu
   || [https://github.com/D-Programming-Language/phobos/pull/1335 Code]
+
   ||
 +
   || [https://github.com/D-Programming-Language/phobos/pull/529 Code]
 +
  || [http://erdani.com/d/web/phobos-prerelease/std_benchmark.html Docs]  
 
   ||  
 
   ||  
   || [http://forum.dlang.org/post/koqrq8$h4b$1@digitalmars.com Discussion]
+
   || <font color=black>Abandoned</font>
  || <font color=black>On hold</font> / <font color=black>Abandoned</font>
 
 
|-
 
|-
 
| dtoh
 
| dtoh
 
   || Adam D. Ruppe
 
   || Adam D. Ruppe
 +
  ||
 
   || [https://github.com/adamdruppe/tools/blob/dtoh/dtoh.d Code]
 
   || [https://github.com/adamdruppe/tools/blob/dtoh/dtoh.d Code]
   ||  
+
   || [http://forum.dlang.org/post/lgspgg$2i8l$1@digitalmars.com RFC Thread]
 
   || [https://github.com/D-Programming-Language/tools/pull/39 Pull request]
 
   || [https://github.com/D-Programming-Language/tools/pull/39 Pull request]
 
   || <font color=black>On hold</font>
 
   || <font color=black>On hold</font>
 +
|-
 +
| std.xml2
 +
  || Michael Rynn
 +
  ||
 +
  || [https://launchpad.net/d2-xml Code]
 +
  || [http://dsource.org/projects/xmlp Docs]
 +
  ||
 +
  || <font color=black>Abandoned</font>
 
|-
 
|-
 
| std.halffloat
 
| std.halffloat
 
   || Walter Bright
 
   || Walter Bright
 +
  ||
 
   || [https://github.com/D-Programming-Language/phobos/pull/2207 Code]
 
   || [https://github.com/D-Programming-Language/phobos/pull/2207 Code]
 
   ||  
 
   ||  
 
   || [https://github.com/D-Programming-Language/phobos/pull/2207 Pull request]
 
   || [https://github.com/D-Programming-Language/phobos/pull/2207 Pull request]
   || <font color=black>Work in progress</font>
+
   || <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
 +
  || Jose Armando Garcia
 +
  ||
 +
  || [https://github.com/D-Programming-Language/phobos/pull/432/files Code]
 +
  || [http://jsancio.github.com/phobos/phobos/std_log.html Docs]
 +
  || [http://forum.dlang.org/post/jhbbfd$1tmk$1@digitalmars.com Review thread]
 +
  || <font color=black>Abandoned</font>
 
|}
 
|}
  
Line 116: 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
 
| 2014-04-08

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 ? ?