[swift-build-dev] [swift-evolution] [swift-evolution-announce] [Review] SE-0150 Package Manager Support for branches

[T.J. Usiyan]

  • What is your evaluation of the proposal?
+ 1
        • Is the problem being addressed significant enough to warrant a
change to Swift?
Yes
        • Does this proposal fit well with the feel and direction of Swift?
Yes
        • If you have used other languages or libraries with a similar
feature, how do you feel that this proposal compares to those?
Yes. It's about the same.
        • How much effort did you put into your review? A glance, a quick
reading, or an in-depth study?
More information about the Swift evolution process is available at
A quick read.

On Mon, Jan 30, 2017 at 3:41 PM, David Sweeris via swift-evolution <
swift-evolution at swift.org> wrote:

>
> > On Jan 24, 2017, at 10:18 AM, Daniel Dunbar <daniel_dunbar at apple.com>
> wrote:
> >
> > Hello Swift community,
> >
> > The review of SE-0150 “ Package Manager Support for branches" begins now
> and runs through January 31, 2017. The proposal is available here:
> >
> >  https://github.com/apple/swift-evolution/blob/master/
> proposals/0150-package-manager-branch-support.md
> >
> > Reviews are an important part of the Swift evolution process. All
> reviews should be sent to the swift-build-dev and swift-evolution mailing
> lists at
> >  https://lists.swift.org/mailman/listinfo/swift-build-dev
> >  https://lists.swift.org/mailman/listinfo/swift-evolution
> > or, if you would like to keep your feedback private, directly to the
> review manager. When replying, please try to keep the proposal link at the
> top of the message:
> >  https://github.com/apple/swift-evolution/blob/master/
> proposals/0150-package-manager-branch-support.md
> >
> > What goes into a review?
> >
> > The goal of the review process is to improve the proposal under review
> through constructive criticism and, eventually, determine the direction of
> Swift. When writing your review, here are some questions you might want to
> answer in your review:
> >
> >       • What is your evaluation of the proposal?
> +1
> I would even go a bit further and suggest that "Package(url: String,
> branch: String)” should have “master” be the default value for `branch`, to
> make it simpler for small developer teams who don’t necessarily have all
> the collaboration issues of large projects.
>
> >       • Is the problem being addressed significant enough to warrant a
> change to Swift?
> IMHO, yes. Particularly since Xcode doesn’t support git tags (that I can
> find, anyway)
>
> >       • Does this proposal fit well with the feel and direction of Swift?
> IMOH, yes. This proposal will make development with much easier for
> programmers whose work-flow is branch-based rather than tag-based.
>
> >       • How much effort did you put into your review? A glance, a quick
> reading, or an in-depth study?
> An embarrassingly long time trying to figure out why a package wasn’t
> seeing changes I’d committed to one of its dependencies... Somewhere along
> the line I must’ve accidentally created a branch called “1.0.0”, which lead
> to me conflating the 1.0.0 tag with the 1.0.0 branch, which has caused me
> no end of head scratching.
>
> - Dave Sweeris
> _______________________________________________
> swift-evolution mailing list
> swift-evolution at swift.org
> https://lists.swift.org/mailman/listinfo/swift-evolution
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-build-dev/attachments/20170130/9512dcee/attachment.html>