I think there are docs on this in the main README or in a CONTRIBUTING file.
Mishal (+CC) knows for sure. Michael > On Aug 9, 2017, at 5:41 AM, David Zarzycki via swift-dev > <[email protected]> wrote: > > Hi Slava, > > Thanks! I appreciate the suggestion to run tests, but what I’ve tried in the > past, @swift-ci ignored me. Is there a whitelist for who can request that > tests be run? That would make sense from a security perspective. > > Dave > > > >> On Aug 8, 2017, at 23:48, Slava Pestov <[email protected] >> <mailto:[email protected]>> wrote: >> >> I see you’re two steps ahead of me already: >> https://github.com/apple/swift/pull/11397 >> <https://github.com/apple/swift/pull/11397> >> >> This looks good, nice catch! >> >> Slava >> >>> On Aug 8, 2017, at 8:45 PM, Slava Pestov <[email protected] >>> <mailto:[email protected]>> wrote: >>> >>> >>>> On Aug 8, 2017, at 3:34 PM, David Zarzycki via swift-dev >>>> <[email protected] <mailto:[email protected]>> wrote: >>>> >>>> In ConstraintSystem::performMemberLookup(), constructors with “simple” >>>> names have a dedicated lookup path. In contrast, constructors with >>>> compound names are handled by the normal lookup. If I delete this code and >>>> let the normal lookup path handle both simple and compound named >>>> constructors, I find that all 10,214 validation tests pass on my machine >>>> (albeit with slightly different error messages in three test files). >>> >>> I would suggest running the source compatibility test suite also (see “pull >>> request testing” in https://swift.org/source-compatibility/ >>> <https://swift.org/source-compatibility/>), but it is quite possible the >>> code is indeed unnecessary. >>> >>>> >>>> Is the test suite missing a test for this code path and if so, what? Or >>>> should it be scheduled for deletion after identical error messages can be >>>> generated by the normal lookup path? >>> >>> Are the new error messages worse or just different? If the latter there’s >>> really no requirement to keep them identical. >>> >>> Slava >>> >>>> >>>> Dave >>>> _______________________________________________ >>>> swift-dev mailing list >>>> [email protected] <mailto:[email protected]> >>>> https://lists.swift.org/mailman/listinfo/swift-dev >>>> <https://lists.swift.org/mailman/listinfo/swift-dev> >>> >> > > _______________________________________________ > swift-dev mailing list > [email protected] > https://lists.swift.org/mailman/listinfo/swift-dev
_______________________________________________ swift-dev mailing list [email protected] https://lists.swift.org/mailman/listinfo/swift-dev
