-
Notifications
You must be signed in to change notification settings - Fork 486
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
Add ventura to mac tests configurations #17349
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Here is the bulk of the test failures reported by the Ventura bots: https://gist.github.com/dalexsoto/70412a0bb98d5794783827700018f49c |
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.
Finally Ventura test results! But we'll need to fix the test failures before we can merge the PR.
This comment has been minimized.
This comment has been minimized.
…d build-pull-request.
f9b0876
to
91c904c
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
yaml code is perfect, but as already mentioned, we need to fix the tests and then land this.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
…e protocols they do not conform to to ApiProtocolTest
Note: Introspection tests aren't working in the test runner for me so need to fix that. So apologies if something is blatantly wrong... I'm working towards getting all the tests to run now. But please let me know if my edits aren't correct for the various tests. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@rolfbjarne / @mandel-macaque I think this is ready for review |
Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>
Thanks @rolfbjarne, suggestions are applied. |
✅ API diff for current PR / commitLegacy Xamarin (No breaking changes)
NET (empty diffs)
✅ API diff vs stableLegacy Xamarin (No breaking changes).NET (No breaking changes)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
💻 [PR Build] Tests on macOS M1 - Mac Ventura (13.0) passed 💻✅ All tests on macOS M1 - Mac Ventura (13.0) passed. Pipeline on Agent |
💻 [PR Build] Tests on macOS M1 - Mac Big Sur (11.5) passed 💻✅ All tests on macOS M1 - Mac Big Sur (11.5) passed. Pipeline on Agent |
🚀 [CI Build] Test results 🚀Test results✅ All tests passed on VSTS: simulator tests. 🎉 All 225 tests passed 🎉 Tests counts✅ bcl: All 69 tests passed. Html Report (VSDrops) Download Pipeline on Agent |
Added Ventura machines to macTestConfigurations within both the build-ci-pipeline and the build-pr-pipelines. --------- Co-authored-by: Alex Soto <alex@alexsoto.me> Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>
Added Ventura machines to macTestConfigurations within both the build-ci-pipeline and the build-pr-pipelines. This is a backport of #xamarin#17349.
Added Ventura machines to macTestConfigurations within both the build-ci-pipeline and the build-pr-pipelines. This is a backport of #xamarin#17349.
Added Ventura machines to macTestConfigurations within both the build-ci-pipeline and the build-pr-pipelines. This is a backport of #xamarin#17349.
…17816) Added Ventura machines to macTestConfigurations within both the build-ci-pipeline and the build-pr-pipelines. This is a backport of #17349, #16743, and #16777. --------- Co-authored-by: dustin-wojciechowski <dustin.wojciechowski@microsoft.com> Co-authored-by: GitHub Actions Autoformatter <github-actions-autoformatter@xamarin.com>
Added Ventura machines to macTestConfigurations within both the build-ci-pipeline and the build-pr-pipelines. This is a backport of #17349, #16743, and #16777. --------- Co-authored-by: dustin-wojciechowski <dustin.wojciechowski@microsoft.com> Co-authored-by: GitHub Actions Autoformatter <github-actions-autoformatter@xamarin.com>
…17817) Added Ventura machines to macTestConfigurations within both the build-ci-pipeline and the build-pr-pipelines. This is a backport of #17349, #16743, and #16777. --------- Co-authored-by: dustin-wojciechowski <dustin.wojciechowski@microsoft.com> Co-authored-by: GitHub Actions Autoformatter <github-actions-autoformatter@xamarin.com>
Added Ventura machines to macTestConfigurations within both the build-ci-pipeline and the build-pr-pipelines. This is a backport of xamarin#17349. --------- Co-authored-by: Alex Soto <alex@alexsoto.me> Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>
Added Ventura machines to macTestConfigurations within both the build-ci-pipeline and the build-pr-pipelines. This is a backport of #17349. --------- Co-authored-by: dustin-wojciechowski <dustin.wojciechowski@microsoft.com> Co-authored-by: Alex Soto <alex@alexsoto.me> Co-authored-by: GitHub Actions Autoformatter <github-actions-autoformatter@xamarin.com>
Added Ventura machines to macTestConfigurations within both the build-ci-pipeline and the build-pr-pipelines.