Skip to content

Conversation

@tomerd
Copy link
Contributor

@tomerd tomerd commented Nov 30, 2023

5.10 cherry-pick of #7079

motivation: in some edge cases, the local repository may be partially
valid (the containing directory is a legit git repo, but the repository
directory is not), or otherwise point to a remote different than the one
expected

changes:
* validate that the local repository remote aligns with the expected
one, not just that the directory is a valid git repo
* refactor validation flow to be more streamlined
@tomerd tomerd added the swift 5.10 This PR targets the 5.10 branch label Nov 30, 2023
@tomerd
Copy link
Contributor Author

tomerd commented Nov 30, 2023

@swift-ci test

@tomerd tomerd enabled auto-merge (squash) November 30, 2023 18:38
@tomerd tomerd disabled auto-merge November 30, 2023 18:38
@tomerd tomerd enabled auto-merge (squash) November 30, 2023 18:38
@tomerd tomerd disabled auto-merge November 30, 2023 18:38
@tomerd tomerd enabled auto-merge (rebase) November 30, 2023 18:39
@tomerd
Copy link
Contributor Author

tomerd commented Dec 1, 2023

@compnerd windows failure seem unrelated. are yo aware?

@tomerd
Copy link
Contributor Author

tomerd commented Dec 1, 2023

@swift-ci test windows

@compnerd
Copy link
Member

compnerd commented Dec 1, 2023

@tomerd the link is gone, do you have the logs?

@MaxDesiatov
Copy link
Contributor

@swift-ci test windows

@tomerd tomerd merged commit 7643b60 into swiftlang:release/5.10 Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

swift 5.10 This PR targets the 5.10 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants