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)
 
(24 intermediate revisions by 5 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" | Review Manager
 +
! scope="col" class="unsortable" | Code
 +
! scope="col" class="unsortable" | Documentation
 +
! scope="col" class="unsortable" | Other
 +
! scope="col" | Status
 +
|-
 +
| 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" | Code
 
! scope="col" class="unsortable" | Documentation
 
! scope="col" class="unsortable" | Documentation
Line 21: Line 48:
 
   || [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=darkorange>Ready for voting</font>
+
   || <font color=red>[https://github.com/Hackerpilot/Dscanner/issues/316 Needs work]</font>
 
|-
 
|-
 
| std.decimal
 
| std.decimal
Line 29: Line 56:
 
   || [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]]
Line 48: 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]
  ||
 
 
   ||  
 
   ||  
   || <font color=black>Work in progress</font>
+
  || [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.xml2
+
| std.experimental.si
   || Michael Rynn
+
   || David Nadlinger, Per Nordlöw
 
   ||
 
   ||
   || [https://launchpad.net/d2-xml Code]
+
   || [https://github.com/nordlow/units-d Code]
  || [http://dsource.org/projects/xmlp Docs]  
 
 
   ||
 
   ||
   || <font color=black>On hold</font> / <font color=black>Abandoned</font>
+
  || [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
 +
  ||
 +
  || [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
 +
  ||
 +
  || [https://github.com/D-Programming-Language/phobos/pull/3625 Code/Pull Request]
 +
  || [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"
 
|-
 
|-
| std.log
+
! scope="col" | Package
  || Jose Armando Garcia
+
! scope="col" | Champion
  ||
+
! scope="col" | Review Manager
  || [https://github.com/D-Programming-Language/phobos/pull/432/files Code]
+
! scope="col" class="unsortable" | Code
  || [http://jsancio.github.com/phobos/phobos/std_log.html Docs]
+
! scope="col" class="unsortable" | Documentation
  || [http://forum.dlang.org/post/jhbbfd$1tmk$1@digitalmars.com Review thread]
+
! scope="col" class="unsortable" | Other
  || <font color=black>On hold</font> / <font color=black>Abandoned</font>
+
! 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
Line 86: Line 135:
 
   ||
 
   ||
 
   || [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
Line 96: Line 153:
 
   ||  
 
   ||  
 
   || [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.data.json
+
| std.compression.lz77
   || Sönke Ludwig
+
   || Walter Bright
   || Atila Neves
+
   ||
  || [https://github.com/s-ludwig/std_data_json Code]
+
   || [https://github.com/D-Programming-Language/phobos/pull/1335 Code]
  || [https://s-ludwig.github.io/std_data_json Docs]
 
  || [http://forum.dlang.org/post/lt5s76$is$1@digitalmars.com RFC Thread]
 
[http://forum.dlang.org/post/nbuhouhimowvcqssvfnr@forum.dlang.org Review Thread]
 
  || <font color=darkorange>Ready for comments</font>
 
|-
 
| std.experimental.testing
 
  || Atila Neves
 
  || Robert burner Schadek
 
   || [https://github.com/D-Programming-Language/phobos/pull/3207 Code/Pull Request]
 
  || [https://github.com/D-Programming-Language/phobos/pull/3207 Docs]
 
  || [http://forum.dlang.org/post/uzocokshugchescbawlj@forum.dlang.org Forum thread]
 
[http://forum.dlang.org/post/stbdckpfsysjtppldmry@forum.dlang.org Review Thread]
 
  || <font color=darkorange>Ready for comments</font>
 
|-
 
| std.experimental.safeint
 
  || Robert burner Schadek
 
 
   ||  
 
   ||  
  || [https://github.com/D-Programming-Language/phobos/pull/3389 Code/Pull Request]
+
   || [http://forum.dlang.org/post/koqrq8$h4b$1@digitalmars.com Discussion]
   || [http://burner.github.io/phobos/phobos-prerelease/std_experimental_safeint.html Docs]
+
   || <font color=black>Abandoned</font>
  || [ Forum thread]
 
   || <font color=darkorange>Ready for comments</font>
 
 
|-
 
|-
| std.experimental.ndslice
+
| std.log
   || Ilya Yaroshenko
+
   || Jose Armando Garcia
   || Jack Stouffer
+
   ||
   || [https://github.com/D-Programming-Language/phobos/pull/3397 Code/Pull Request]
+
   || [https://github.com/D-Programming-Language/phobos/pull/432/files Code]
   || [http://dtest.thecybershadow.net/artifact/website-13cbdcf17d84fc31328c3f517a56bea783c418d6-dd2292a424959b594956eeeba64d391f/web/phobos-prerelease/std_experimental_ndslice.html Docs]
+
   || [http://jsancio.github.com/phobos/phobos/std_log.html Docs]  
   || [http://forum.dlang.org/post/uesnmkgniumswfclwbgt@forum.dlang.org Review Thread]
+
   || [http://forum.dlang.org/post/jhbbfd$1tmk$1@digitalmars.com Review thread]
  || <font color=darkorange> Ready for voting </font>
+
   || <font color=black>Abandoned</font>
|-
 
| std.experimental.perpetual
 
  || Sergei Degtiarev
 
  ||
 
  || [https://github.com/D-Programming-Language/phobos/pull/3625 Code/Pull Request]
 
  || [https://sdegtiarev.github.io/ Docs]
 
  || [http://forum.dlang.org/post/euxiznukswgnpselleyi@forum.dlang.org Forum Thread]
 
   || <font color=darkorange>Ready for comments</font>
 
 
|}
 
|}
  
Line 154: 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
 
| 2015-07-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 ? ?