Skip to content
This repository was archived by the owner on Apr 23, 2025. It is now read-only.

Pin swift-benchmark to a specific revision. #640

Merged
merged 1 commit into from
Jul 30, 2020

Conversation

saeta
Copy link
Contributor

@saeta saeta commented Jul 20, 2020

Swift-Benchmark currently has an unstable API. This change pins
Swift-Benchmark to an explicit revision to ensure stable builds.

Swift-Benchmark currently has an unstable API. This change pins
Swift-Benchmark to an explicit revision to ensure stable builds.
@saeta saeta requested a review from shabalind July 20, 2020 21:48
@shabalind
Copy link
Contributor

Is this a duplicate of #641 ?

@shabalind
Copy link
Contributor

shabalind commented Jul 21, 2020

Edit: It's not, the two PRs are for different branches (master vs tensorflow-0.11).

@BradLarson
Copy link
Contributor

@shabalind - Just checking on the status, did we need to modify this to bring it in?

@shabalind
Copy link
Contributor

@BradLarson Changes here look good to me! We should probably do a 0.1 release on swift-benchmark some time soon to make it a bit more user friendly than a particular commit hash.

@saeta
Copy link
Contributor Author

saeta commented Jul 30, 2020

+1 to doing a 0.1 release. Things seem reasonably stable presently. Thanks!

@saeta saeta merged commit 0874d7b into master Jul 30, 2020
@saeta saeta deleted the explicit-swift-benchmark-revision branch July 30, 2020 12:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants