Skip to content

Conversation

adamsitnik
Copy link
Member

@adamsitnik adamsitnik commented Jan 16, 2025

Backport of #110550 and #111208 to release/9.0-staging

Customer Impact

  • Customer reported
  • Found internally

CI only changes.

Regression

  • Yes
  • No

Testing

Both fixes have been tested in the main branch.

Risk

Very low: for #110550 we just perform fewer test cases for large inputs, for #111208 we simply don't test Drawing-related types for Mono as it's not supported.

* don't run Drawing-related tests that do things like creating Bitmaps on Mono, as it's not supported (it does not support ComWrappers)

* re-enable the tests
@adamsitnik adamsitnik added test-enhancement Improvements of test source code binaryformatter-migration Issues related to the removal of BinaryFormatter and migrations away from it labels Jan 16, 2025
@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jan 16, 2025
@adamsitnik adamsitnik added area-System.Formats.Nrbf and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jan 16, 2025
@jkotas jkotas added the Servicing-approved Approved for servicing release label Jan 17, 2025
@jkotas
Copy link
Member

jkotas commented Jan 17, 2025

Test fixes - tell mode.

@jkotas jkotas merged commit f9ffbe0 into dotnet:release/9.0-staging Jan 17, 2025
79 of 91 checks passed
@matouskozak
Copy link
Member

Thank you @adamsitnik for the backport

@github-actions github-actions bot locked and limited conversation to collaborators Feb 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Formats.Nrbf binaryformatter-migration Issues related to the removal of BinaryFormatter and migrations away from it Servicing-approved Approved for servicing release test-enhancement Improvements of test source code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants