-
Notifications
You must be signed in to change notification settings - Fork 121
Add request to get preferred run destination for a platform #760
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
2543368
to
64da735
Compare
64da735
to
c1e0629
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but we should get @jakepetroules's opinion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems reasonable to me, if the API contract is basically "give me a suggested destination given the specified input parameters", without defining/promising the semantics used to define that.
That’s the idea. Do you think this needs to be clarified somewhere in a doc comment or is it clean enough from the current ones? |
c1e0629
to
17b6273
Compare
Added a few test cases as well based on Owen’s comment here: #761. Would be good to get a review for those as well. |
@swift-ci Please test |
@swift-ci test |
17b6273
to
87a881d
Compare
This can be useful when implementing a system that needs to configure the active run destination of a build request but only knows the platform that the user is requesting a build for, but no architecture etc.
87a881d
to
461ae49
Compare
@swift-ci Please test |
This can be useful when implementing a system that needs to configure the active run destination of a build request but only knows the platform that the user is requesting a build for, but no architecture etc.