Skip to content

[Protocol] Enable nullability and fix a few issues. - #13280

Merged
rolfbjarne merged 3 commits into
dotnet:mainfrom
rolfbjarne:nullability-protocol
Nov 8, 2021
Merged

[Protocol] Enable nullability and fix a few issues.#13280
rolfbjarne merged 3 commits into
dotnet:mainfrom
rolfbjarne:nullability-protocol

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member

No description provided.

@rolfbjarne rolfbjarne added the not-notes-worthy Ignore for release notes label Nov 4, 2021
@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff
View dotnet API diff
View dotnet legacy API diff
View dotnet iOS-MacCatalayst API diff

API & Generator diff

ℹ️ API Diff (from PR only) (please review changes)
Generator Diff (only version changes)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

9 tests failed, 90 tests passed.

Failed tests

  • monotouch-test/Mac [dotnet]/Debug [dotnet]: BuildFailure
  • monotouch-test/Mac Catalyst [dotnet]/Debug [dotnet]: BuildFailure
  • monotouch-test/watchOS 32-bits - simulator/Debug (LinkSdk): TimedOut
  • monotouch-test/watchOS 32-bits - simulator/Debug (static registrar): Crashed
  • monotouch-test/watchOS 32-bits - simulator/Release (all optimizations): Crashed
  • xammac tests/Mac Modern/Debug: Failed (Test run failed.
    Tests run: 2544 Passed: 2423 Inconclusive: 10 Failed: 1 Ignored: 120)
  • xammac tests/Mac Modern/Release: Failed (Test run failed.
    Tests run: 2541 Passed: 2419 Inconclusive: 10 Failed: 1 Ignored: 121)
  • xammac tests/Mac Modern/Release (all optimizations): Failed (Test run failed.
    Tests run: 2541 Passed: 2418 Inconclusive: 10 Failed: 1 Ignored: 122)
  • Xtro/Mac: BuildFailure

Pipeline on Agent XAMBOT-1104.BigSur'
Merge c625202 into 696e7a5

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

❌ [CI Build] Tests didn't execute on Build. ❌

Tests didn't execute on Build.

No test summary was found (something probably failed before the tests could execute)

Pipeline on Agent
Merge c934628 into 6b3ebd2

@rolfbjarne

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff
View dotnet API diff
View dotnet legacy API diff
View dotnet iOS-MacCatalayst API diff

API & Generator diff

ℹ️ API Diff (from PR only) (please review changes)
Generator Diff (only version changes)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

5 tests failed, 94 tests passed.

Failed tests

  • monotouch-test/Mac [dotnet]/Debug [dotnet]: Failed (Test run failed.
    Tests run: 2539 Passed: 2413 Inconclusive: 10 Failed: 1 Ignored: 125)
  • monotouch-test/Mac Catalyst [dotnet]/Debug [dotnet]: Failed (Tests run: 2704 Passed: 2500 Inconclusive: 35 Failed: 1 Ignored: 203)
  • xammac tests/Mac Modern/Debug: Failed (Test run failed.
    Tests run: 2544 Passed: 2423 Inconclusive: 10 Failed: 1 Ignored: 120)
  • xammac tests/Mac Modern/Release: Failed (Test run failed.
    Tests run: 2541 Passed: 2419 Inconclusive: 10 Failed: 1 Ignored: 121)
  • xammac tests/Mac Modern/Release (all optimizations): Failed (Test run failed.
    Tests run: 2541 Passed: 2418 Inconclusive: 10 Failed: 1 Ignored: 122)

Pipeline on Agent XAMBOT-1104.BigSur'
Merge c934628 into 6b3ebd2

@rolfbjarne

Copy link
Copy Markdown
Member Author

Test failures are unrelated (https://github.com/xamarin/maccore/issues/2525).

Build is currently broken in main, waiting to merge until that's fixed (#13304).

@rolfbjarne
rolfbjarne merged commit 1fab379 into dotnet:main Nov 8, 2021
@rolfbjarne
rolfbjarne deleted the nullability-protocol branch November 8, 2021 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-notes-worthy Ignore for release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants