Difference between revisions of "Review Queue"

From D Wiki
Jump to: navigation, search
(Moved color to review queue on advice from jpf91)
 
(56 intermediate revisions by 14 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"
 +
|-
 +
! 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.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"
 
{| class="wikitable sortable" border="1"
Line 9: Line 36:
 
! 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 16: Line 44:
 
| [[Review/std.lexer|std.lexer]]
 
| [[Review/std.lexer|std.lexer]]
 
   || Brian Schott
 
   || Brian Schott
   || [https://github.com/Hackerpilot/Dscanner/blob/master/stdx/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.buffer.scopebuffer]]
 
  || Walter Bright
 
  || [https://github.com/D-Programming-Language/phobos/pull/1911 Code]
 
  ||
 
  || [http://forum.dlang.org/post/ld2586$17f6$1@digitalmars.com Announcement]
 
  || <font color=red>Under review</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/decimal Code]
+
  ||
   || [https://github.com/andersonpd/decimal/wiki Docs]  
+
   || [https://github.com/andersonpd/eris Code]
 +
   || [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 57: 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=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.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>
 
   || <font color=black>Work in progress</font>
 
|-
 
|-
| [[Review/std.logger|std.logger]]
+
| std.experimental.safeint
   || Robert Schadek
+
   || Robert burner Schadek
   || [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/D-Programming-Language/phobos/pull/3389 Code/Pull Request]
   || [http://forum.dlang.org/post/mailman.313.1377180809.1719.digitalmars-d@puremagic.com Discussion thread]
+
   || [http://burner.github.io/phobos/phobos-prerelease/std_experimental_safeint.html Docs]
[http://forum.dlang.org/post/wuahsrjdcmdifnfgjfgs@forum.dlang.org Review thread 1]
+
   || [http://forum.dlang.org/post/ulaywuujlhhubxiidagw@forum.dlang.org Forum thread]
   || <font color=black>Work in progress</font>
+
  || <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.io
+
| std.experimental.perpetual
   || Steven Schveighoffer
+
   || Sergei Degtiarev
  || [https://github.com/schveiguy/phobos/blob/new-io/std/io.d Code]
 
 
   ||  
 
   ||  
 +
  || [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"
 +
|-
 +
! 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.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>
+
   || <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>Work in progress</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=black>On hold</font> / <font color=black>Abandoned</font>
+
   || <font color=black>Abandoned</font>
 
|-
 
|-
| std.log
+
| std.halffloat
   || Jose Armando Garcia
+
  || Walter Bright
   || [https://github.com/D-Programming-Language/phobos/pull/432/files Code]
+
   ||
   || [http://jsancio.github.com/phobos/phobos/std_log.html Docs]
+
   || [https://github.com/D-Programming-Language/phobos/pull/2207 Code]
  || [http://forum.dlang.org/post/jhbbfd$1tmk$1@digitalmars.com Review thread]
+
  ||
   || <font color=black>On hold</font> / <font color=black>Abandoned</font>
+
   || [https://github.com/D-Programming-Language/phobos/pull/2207 Pull request]
 +
   || <font color=black>On hold</font>
 
|-
 
|-
 
| std.compression.lz77
 
| std.compression.lz77
 
   || Walter Bright
 
   || Walter Bright
 +
  ||
 
   || [https://github.com/D-Programming-Language/phobos/pull/1335 Code]
 
   || [https://github.com/D-Programming-Language/phobos/pull/1335 Code]
 
   ||  
 
   ||  
 
   || [http://forum.dlang.org/post/koqrq8$h4b$1@digitalmars.com Discussion]
 
   || [http://forum.dlang.org/post/koqrq8$h4b$1@digitalmars.com Discussion]
   || <font color=black>On hold</font> / <font color=black>Abandoned</font>
+
   || <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 117: 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  
 
| 2014-01-20  

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