<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Mar 7, 2017, at 12:53 PM, Rick Ballard via swift-build-dev &lt;<a href="mailto:swift-build-dev@swift.org" class="">swift-build-dev@swift.org</a>&gt; wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html charset=us-ascii" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hello Swift community,<br class=""><br class="">The review of "SE-0158 Package Manager Manifest API Redesign" begins now and runs through <b class="">March 13, 2017</b>.&nbsp;The proposal is available here:<br class=""><br class=""><span class="Apple-tab-span" style="white-space:pre">        </span><a href="https://github.com/apple/swift-evolution/blob/master/proposals/0158-package-manager-manifest-api-redesign.md" class="">https://github.com/apple/swift-evolution/blob/master/proposals/0158-package-manager-manifest-api-redesign.md</a><br class=""><br class="">Reviews are an important part of the Swift evolution process. All Swift Package Manager reviews&nbsp;should be sent to the swift-evolution and swift-build-dev mailing lists at<br class=""><br class=""><span class="Apple-tab-span" style="white-space:pre">        </span><a href="https://lists.swift.org/mailman/listinfo/swift-evolution" class="">https://lists.swift.org/mailman/listinfo/swift-evolution</a><br class=""><span class="Apple-tab-span" style="white-space:pre">        </span>https://lists.swift.org/mailman/listinfo/swift-build-dev<div class=""><br class="">or, if you would like to keep your feedback private, directly to the&nbsp;review manager. When replying, please try to keep the proposal link at&nbsp;the top of the message:<br class=""><br class="">&gt; Proposal link:<br class="">&gt;&gt; &nbsp;https://github.com/apple/swift-evolution/blob/master/proposals/0158-package-manager-manifest-api-redesign.md<br class=""><br class="">&gt; &nbsp;Reply text<br class=""><br class="">&gt;&gt; &nbsp;Other replies<br class=""><br class=""><b class="">What goes into a review?</b><br class=""><br class="">The goal of the review process is to improve the proposal under review&nbsp;through constructive criticism and, eventually, determine the direction of&nbsp;Swift. When writing your review, here are some questions you might want to&nbsp;answer in your review:<br class=""><br class="">* What is your evaluation of the proposal?<br class=""></div></div></div></blockquote><br class=""><div>+1, although I don’t know why we're supporting this:</div><div><pre style="box-sizing: border-box; font-family: SFMono-Regular, Consolas, 'Liberation Mono', Menlo, Courier, monospace; font-size: 13.600000381469727px; margin-top: 0px; margin-bottom: 0px; line-height: 1.45; word-wrap: normal; padding: 16px; overflow: auto; background-color: rgb(246, 248, 250); border-top-left-radius: 3px; border-top-right-radius: 3px; border-bottom-right-radius: 3px; border-bottom-left-radius: 3px; word-break: normal; color: rgb(36, 41, 46);" class=""><span class="pl-c" style="box-sizing: border-box; color: rgb(150, 152, 150);"><span class="pl-c" style="box-sizing: border-box;">//</span> 1.5.8 ..&lt; 2.0.0</span>
<span class="pl-c" style="box-sizing: border-box; color: rgb(150, 152, 150);"></span>.<span class="pl-c1" style="box-sizing: border-box; color: rgb(0, 134, 179);">package</span>(<span class="pl-c1" style="box-sizing: border-box; color: rgb(0, 134, 179);">url</span>: <span class="pl-s" style="box-sizing: border-box; color: rgb(24, 54, 145);"><span class="pl-pds" style="box-sizing: border-box;">"</span>/SwiftyJSON<span class="pl-pds" style="box-sizing: border-box;">"</span></span>, <span class="pl-c1" style="box-sizing: border-box; color: rgb(0, 134, 179);">from</span>: <span class="pl-s" style="box-sizing: border-box; color: rgb(24, 54, 145);"><span class="pl-pds" style="box-sizing: border-box;">"</span>1.5.8<span class="pl-pds" style="box-sizing: border-box;">"</span></span>),</pre><div class="">when, at least as far as I can tell, this:</div><div class=""><pre style="box-sizing: border-box; font-family: SFMono-Regular, Consolas, 'Liberation Mono', Menlo, Courier, monospace; font-size: 13.600000381469727px; margin-top: 0px; margin-bottom: 0px; line-height: 1.45; word-wrap: normal; padding: 16px; overflow: auto; background-color: rgb(246, 248, 250); border-top-left-radius: 3px; border-top-right-radius: 3px; border-bottom-right-radius: 3px; border-bottom-left-radius: 3px; word-break: normal; color: rgb(36, 41, 46);" class=""><span class="pl-c" style="box-sizing: border-box; color: rgb(150, 152, 150);"><span class="pl-c" style="box-sizing: border-box;">//</span> 1.5.8 ..&lt; 2.0.0</span>
<span class="pl-c" style="box-sizing: border-box; color: rgb(150, 152, 150);"></span>.<span class="pl-c1" style="box-sizing: border-box; color: rgb(0, 134, 179);">package</span>(<span class="pl-c1" style="box-sizing: border-box; color: rgb(0, 134, 179);">url</span>: <span class="pl-s" style="box-sizing: border-box; color: rgb(24, 54, 145);"><span class="pl-pds" style="box-sizing: border-box;">"</span>/SwiftyJSON<span class="pl-pds" style="box-sizing: border-box;">"</span></span>, .<span class="pl-c1" style="box-sizing: border-box; color: rgb(0, 134, 179);">uptoNextMajor</span>(<span class="pl-s" style="box-sizing: border-box; color: rgb(24, 54, 145);"><span class="pl-pds" style="box-sizing: border-box;">"</span>1.5.8<span class="pl-pds" style="box-sizing: border-box;">"</span></span>)),</pre><div class="">does the same thing, and the spelling is, at least to me, clearer as well. Dunno, maybe the “from” version is a term of art that I’m just not familiar with.</div></div></div><div><br class=""></div></div><div><blockquote type="cite" class=""><div class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">* Is the problem being addressed significant enough to warrant a&nbsp;change to the Swift Package Manager?<br class=""></div></div></div></blockquote><br class="">Yep</div><div><br class=""><blockquote type="cite" class=""><div class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">* Does this proposal fit well with the feel and direction of Swift?<br class=""></div></div></div></blockquote><br class="">Yep</div><div><br class=""><blockquote type="cite" class=""><div class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">* If you have used other languages, libraries, or package managers with a similar&nbsp;feature, how do you feel that this proposal compares to those?<br class=""></div></div></div></blockquote><br class="">N/A</div><div><br class=""><blockquote type="cite" class=""><div class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">* How much effort did you put into your review? A glance, a quick&nbsp;reading, or an in-depth study?<br class=""></div></div></div></blockquote><br class=""><div>Read the discussion thread and the proposal a few times.</div></div><br class=""><div class="">- Dave Sweeris</div></body></html>