Difference between revisions of "Review Queue"

From D Wiki
Jump to: navigation, search
(Moved color to review queue on advice from jpf91)
 
(135 intermediate revisions by 24 users not shown)
Line 1: Line 1:
Current proposals that are up for review to be included into Phobos
+
This is the review queue of pending packages to get into the D Standard Library ([[Phobos]]). The review process is explained [[Review/Process|here]].
  
== Proposals ==
+
If you want a project to put on this list, please look at the [[wish list]].
  
{| class="wikitable"
+
== 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"
 +
|-
 +
! 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
 +
  ||
 +
  || [https://github.com/jacob-carlborg/phobos/tree/serialization Code]
 +
  || [https://dl.dropboxusercontent.com/u/18386187/docs/std.serialization/index.html  Docs]
 +
  || [https://github.com/jacob-carlborg/orange/wiki/_pages Examples]
 +
[http://forum.dlang.org/post/adyanbsdsxsfdpvoozne@forum.dlang.org Review thread 1]
 +
 
 +
[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>
 
|-
 
|-
! scope="col"| Proposal
+
| std.experimental.safeint
! scope="col"| Github
+
  || Robert burner Schadek
! scope="col"| Champion
+
  ||
! scope="col"| Status*
+
  || [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>
 
|-
 
|-
| [[Review/std.process]]
+
| std.experimental.checkedint
| https://github.com/alexrp/phobos/tree/new-std-process-update
+
  || Thomas Stuart Bockman aka. tsbockmann  ||
| Alex Rønne Petersen
+
  || [https://github.com/dlang/phobos/pull/4407 Code/Pull Request]
| <font color=darkorange>pending for review</font>
+
  || [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.
  
Status
+
{| class="wikitable sortable" border="1"
** <font color=darkorange>pending for review</font>
+
|-
** <font color=cornflowerblue>under review</font>
+
! scope="col" | Package
** <font color=green>accepted</font>
+
! scope="col" | Champion
** <font color=red>rejected</font>
+
! 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>Abandoned</font>
 +
|-
 +
| dtoh
 +
  || Adam D. Ruppe
 +
  ||
 +
  || [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]
 +
  || <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
 +
  || Walter Bright
 +
  ||
 +
  || [https://github.com/D-Programming-Language/phobos/pull/2207 Code]
 +
  ||
 +
  || [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
 +
  || 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>
 +
|}
 +
 
 +
== Previously Reviewed ==
 +
 
 +
{| class="wikitable sortable" border="1"
 +
|-
 +
! scope="col" | Date
 +
! scope="col" | Module
 +
! scope="col" | Description
 +
! scope="col" | Author
 +
! scope="col" | Accepted
 +
! scope="col" | Votes (yes/no)
 +
! scope="col" | Review Manager
 +
! scope="col" | Date included in Phobos
 +
! 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
 +
  || std.uni
 +
  || More complete API
 +
  || Dmitry Olshansky
 +
  |bgcolor=limegreen| Yes
 +
  || 15/0
 +
  || Jesse Phillips
 +
  || 2013-08-08
 +
  || [https://github.com/D-Programming-Language/phobos/commit/c109594456fb929447741ade09928b3cc679069e c1095944]
 +
|-
 +
| 2013-04-20
 +
  || std.process
 +
  || Revamped API
 +
  || Lars T. Kyllingstad / Steven Schveighoffer
 +
  |bgcolor=limegreen| Yes
 +
  || 18/0
 +
  || Jesse Phillips
 +
  || 2013-05-07
 +
  || [https://github.com/D-Programming-Language/phobos/commit/9d987513855ce00440b6f6af255e2a95ab8eab9a 9d987513]
 +
|-
 +
| 2012-08-29
 +
  || std.digest
 +
  || Digests (SHA; CRC)
 +
  || Johannes Pfau
 +
  |bgcolor=limegreen| Yes
 +
  || 6/1
 +
  || Dmitry Olshansky
 +
  || 2012-09-16
 +
  || [https://github.com/D-Programming-Language/phobos/commit/688e1d40573775e74b7b03d259e9c194746255b8 688e1d40]
 +
|-
 +
| 2012-06-27
 +
  || std.uuid
 +
  || UUIDs
 +
  || Johannes Pfau
 +
  |bgcolor=limegreen| Yes
 +
  || 10/0
 +
  || Dmitry Olshansky
 +
  || 2012-07-02
 +
  || [https://github.com/D-Programming-Language/phobos/commit/8ac3aea10b1ea1f78506ca9d73285032c9f12499 8ac3aea1]
 +
|-
 +
| 2012-03-12
 +
  || std.log
 +
  || Logging
 +
  || José Armando García
 +
  |bgcolor=gold| Suspended
 +
  || -
 +
  || David Nadlinger
 +
  || -
 +
  || -
 +
|-
 +
| 2011-12-29
 +
  || std.net.curl
 +
  || CURL wrapper
 +
  || Jonas Drewsen
 +
  |bgcolor=limegreen| Yes
 +
  || 14/0
 +
  || DSimcha
 +
  || 2012-01-28
 +
  || [https://github.com/D-Programming-Language/phobos/commit/ddff40918b1e69170de6d070d68b82ab854e21c0 ddff4091]
 +
|-
 +
| 2011-11-20
 +
  || std.csv
 +
  || csv handling
 +
  || Jesse Phillips
 +
  |bgcolor=limegreen| Yes
 +
  || 5/1
 +
  || DSimcha
 +
  || ?
 +
  || ?
 +
|-
 +
| 2011-10-08
 +
  || std.regex
 +
  || Regular Expressions
 +
  || Dmitry Olshansky
 +
  |bgcolor=limegreen| Yes
 +
  || 10/0
 +
  || Jesse Phillips
 +
  || 2011-11-06
 +
  || [https://github.com/D-Programming-Language/phobos/commit/a627abd3916d9aeb5e1899bd93fa539a16109f49 a627abd3]
 +
|-
 +
| 2011-09-01
 +
  || std.net.curl
 +
  || CURL wrapper
 +
  || Jonas Drewsen
 +
  |bgcolor=gold| Postponed
 +
  || -
 +
  || David Nadlinger
 +
  || -
 +
  || -
 +
|-
 +
| 2011-08-12
 +
  || std.path
 +
  || Path manipulation
 +
  || Lars Kyllingstad
 +
  |bgcolor=limegreen| Yes
 +
  || 22/0
 +
  || DSimcha
 +
  || ?
 +
  || ?
 +
|}
  
 
[[Category:Review]]
 
[[Category:Review]]
 +
[[Category:Library Development]]

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