[LEAF] Review Results

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[LEAF] Review Results

Boost - Dev mailing list
I want to thank Emil for his patience for the publishing of the review
results as life got in my way.

Thank you Emil for submitting LEAF to be reviewed for inclusion into
Boost. As an already-Boost-author, you knew what you were getting into
and continued into the process. I also want to thank you for once again
pushing our thoughts on error handling.

--------
Result
--------

This is the part everybody scrolls to the bottom to read, so let’s just
put it at the top. Boost.LEAF is ACCEPTED into Boost. After the
following modifications are made to the library, the repository can be
transitioned to Boost for inclusion into the next release (with the
release cycle timing constraints):

Specify the TLS requirement sooner in the documentation.
Optional usage of Boost.Config to perform all of the Boost Configy
things. (further discussion on this below)
Macro naming and general Boostification clean-up.


--------
Reviews
--------

There was a good amount of discussion about the library on both the dev
and user mail lists. I want to personally thank those who got involved
in the review. You make the process work!

Result summary:

   Accept: 7
   Reject: 1
   No-decision: 3

The no-decision reviews provided valuable input and discussion threads.
While it would be my preference to have the review authors provide a
final verdict, I was grateful for the time and energy they spent with
the library and formulating thoughts into emails for discussion.

Some comments from reviews:

“I find function preload() really great and novel. … This really makes
the error reporting code brief, clean and bug-free.”

“I like LEAF very much and there's a lot offered here…”

“I like it.”

“...it allows bridging multiple error handling strategies and usage
patterns in an efficient way.”


About the docs:
“They look good and manage to explain hard concepts in a very
approachable fashion.”

“The documentation is great -- in terms of styles it's probably the best
I've seen for a C++ library.”

“Clear, friendly and complete.”


--------------
Some Thoughts
--------------

While there was overwhelming consensus that LEAF should be included into
Boost, the process is not democratic. I have spent considerable time
understanding the concerns raised in each of the reviews and in
particular the singular rejection.

Emil took time responding to each of the reviews. The exchanges were
fruitful and both “conditional accept” and “reject” reviewers changed
their verdicts to accept along the way. Most of the concerns were
amicably resolved. I am comfortable with the design choices and
implementation made by the author for the remaining items.

One reviewer brought up the re-implementation of Boost.Config type
facilities. This of course is a complex issue for a library with zero
dependencies. The ability to simply drop the Boost.LEAF directory into a
project and run is a definite plus for a specific user segment. Being
able to modify a single Boost.Config repository to support
WackyCompiler14 targeting LEG Core and get a bunch of libraries for the
ride is also a plus. Emil and I have discussed these concerns and the
best choice seems to be defaulting to zero dependencies and the option
to pull in Boost.Config to provide configuration type facilities if needed.

Accepting a library into Boost is as much about the library as it is
bringing the author into a group. While Emil is already part of the
author crowd, it was clear to me through his careful, informative, and
patient responses that the addition of Boost.LEAF brings professional
maturity that strengthens the community.

A user with more than a year of experience with the library voted
acceptance based on the “qualities of the library itself and the
responsiveness and openness for feedback of its author.”

Throughout the review period, Emil integrated feedback to both the
documentation and code in another branch. It was wonderful to see the
community provide input and the author respond so quickly to produce a
better final result.

Thank you Emil for your vision of better error handling and striving to
bring us along!


--
Michael Caisse
Ciere Consulting
ciere.com

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
Reply | Threaded
Open this post in threaded view
|

Re: [LEAF] Review Results

Boost - Dev mailing list
Michael, thank you for the thoughtful review! I hope whatever circumstances
interfered to delay it are all sorted out.

I'll make the requested changes and it'll be all ready for inclusion in the
next Boost release.

On Fri, Aug 28, 2020 at 3:37 PM Michael Caisse via Boost <
[hidden email]> wrote:

> I want to thank Emil for his patience for the publishing of the review
> results as life got in my way.
>
> Thank you Emil for submitting LEAF to be reviewed for inclusion into
> Boost. As an already-Boost-author, you knew what you were getting into
> and continued into the process. I also want to thank you for once again
> pushing our thoughts on error handling.
>
> --------
> Result
> --------
>
> This is the part everybody scrolls to the bottom to read, so let’s just
> put it at the top. Boost.LEAF is ACCEPTED into Boost. After the
> following modifications are made to the library, the repository can be
> transitioned to Boost for inclusion into the next release (with the
> release cycle timing constraints):
>
> Specify the TLS requirement sooner in the documentation.
> Optional usage of Boost.Config to perform all of the Boost Configy
> things. (further discussion on this below)
> Macro naming and general Boostification clean-up.
>
>
> --------
> Reviews
> --------
>
> There was a good amount of discussion about the library on both the dev
> and user mail lists. I want to personally thank those who got involved
> in the review. You make the process work!
>
> Result summary:
>
>    Accept: 7
>    Reject: 1
>    No-decision: 3
>
> The no-decision reviews provided valuable input and discussion threads.
> While it would be my preference to have the review authors provide a
> final verdict, I was grateful for the time and energy they spent with
> the library and formulating thoughts into emails for discussion.
>
> Some comments from reviews:
>
> “I find function preload() really great and novel. … This really makes
> the error reporting code brief, clean and bug-free.”
>
> “I like LEAF very much and there's a lot offered here…”
>
> “I like it.”
>
> “...it allows bridging multiple error handling strategies and usage
> patterns in an efficient way.”
>
>
> About the docs:
> “They look good and manage to explain hard concepts in a very
> approachable fashion.”
>
> “The documentation is great -- in terms of styles it's probably the best
> I've seen for a C++ library.”
>
> “Clear, friendly and complete.”
>
>
> --------------
> Some Thoughts
> --------------
>
> While there was overwhelming consensus that LEAF should be included into
> Boost, the process is not democratic. I have spent considerable time
> understanding the concerns raised in each of the reviews and in
> particular the singular rejection.
>
> Emil took time responding to each of the reviews. The exchanges were
> fruitful and both “conditional accept” and “reject” reviewers changed
> their verdicts to accept along the way. Most of the concerns were
> amicably resolved. I am comfortable with the design choices and
> implementation made by the author for the remaining items.
>
> One reviewer brought up the re-implementation of Boost.Config type
> facilities. This of course is a complex issue for a library with zero
> dependencies. The ability to simply drop the Boost.LEAF directory into a
> project and run is a definite plus for a specific user segment. Being
> able to modify a single Boost.Config repository to support
> WackyCompiler14 targeting LEG Core and get a bunch of libraries for the
> ride is also a plus. Emil and I have discussed these concerns and the
> best choice seems to be defaulting to zero dependencies and the option
> to pull in Boost.Config to provide configuration type facilities if needed.
>
> Accepting a library into Boost is as much about the library as it is
> bringing the author into a group. While Emil is already part of the
> author crowd, it was clear to me through his careful, informative, and
> patient responses that the addition of Boost.LEAF brings professional
> maturity that strengthens the community.
>
> A user with more than a year of experience with the library voted
> acceptance based on the “qualities of the library itself and the
> responsiveness and openness for feedback of its author.”
>
> Throughout the review period, Emil integrated feedback to both the
> documentation and code in another branch. It was wonderful to see the
> community provide input and the author respond so quickly to produce a
> better final result.
>
> Thank you Emil for your vision of better error handling and striving to
> bring us along!
>
>
> --
> Michael Caisse
> Ciere Consulting
> ciere.com
>
> _______________________________________________
> Unsubscribe & other changes:
> http://lists.boost.org/mailman/listinfo.cgi/boost
>

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
Reply | Threaded
Open this post in threaded view
|

Re: [LEAF] Review Results

Boost - Dev mailing list
In reply to this post by Boost - Dev mailing list
On Sat, 29 Aug 2020 at 00:37, Michael Caisse via Boost
<[hidden email]> wrote:

>
> I want to thank Emil for his patience for the publishing of the review
> results as life got in my way.
>
> Thank you Emil for submitting LEAF to be reviewed for inclusion into
> Boost. As an already-Boost-author, you knew what you were getting into
> and continued into the process. I also want to thank you for once again
> pushing our thoughts on error handling.
>
> --------
> Result
> --------
>
> This is the part everybody scrolls to the bottom to read, so let’s just
> put it at the top. Boost.LEAF is ACCEPTED into Boost.

Thank you Michael and all reviewers, and congratulations to Emil!

The review results page has been updated.

Best regards,
--
Mateusz Loskot, http://mateusz.loskot.net

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost