Skip to content
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

Community PR test pass 1/4/2021 #3948

Closed
wants to merge 17 commits into from
Closed

Conversation

ryalanms
Copy link
Member

@ryalanms ryalanms commented Jan 5, 2021

Community PR Test Pass 1/4/2021

These PRs will be merged if no regressions are found. These limited test passes take the team about 24 hours.

Community PRs included in this test branch:

Don't double import WinFX targets #2976
Make CustomGuids as readonly #3907
Using readonly struct in WPF Ink module #3908
Replace the Dictionary with HashSet in AppDomainShutdownMonitor
Special the capacity to improve performance #3086
Use the new Marshal.IsTypeVisibleFromCom Api #3279

Test Matrix

OS Architecture
Windows 7 SP1 Enterprise x86
Windows 7 SP1 Enterprise x64
Windows Blue Enterprise x86
Windows Server 2012 R2 SP1 x64
Win10 RS1 Enterprise x86
Win10 RS3 Enterprise x64
Win10 RS4 Enterprise x86
Windows Server 2019 x64
Win10 RS6 Enterprise X86
Windows Server 2016 x64

lindexi and others added 17 commits June 3, 2020 19:50
Respect EditorConfig preferences for whitespace and newlines
The MSBuild items & targets that were defined after the WinFX targets import doesn't depend on anything from the WinFX targets.
So, it is safe and better to declare the import after those items and targets definitions.
When the SDK's implementation is being used.
Currently there's no way to properly switch between NETFX's and CoreCLR's WinFX targets.
This adds an opt-out, just in case, if we want to use the NETFX's WinFX targets instead.
@ryalanms ryalanms requested a review from a team as a code owner January 5, 2021 00:47
@ghost ghost added the PR metadata: Label to tag PRs, to facilitate with triage label Jan 5, 2021
@ghost ghost requested review from fabiant3 and SamBent January 5, 2021 00:47
@ryalanms
Copy link
Member Author

ryalanms commented Jan 7, 2021

Apologies for the delay. We have the test results but there are a few failures that appear to be test infrastructure problems on a few OS images. Once these are investigated and re-run, we will get these PRs in.

Thanks everyone for your contributions! This process should get faster now that the test team has more capacity for WPF.

@ryalanms ryalanms added the * NO MERGE * metadata: The PR is not ready for merge yet (see discussion for detailed reasons) label Jan 12, 2021
@ryalanms
Copy link
Member Author

We should merge individual PRs, not the community test pass branch.

@ryalanms
Copy link
Member Author

All PRs were merged individually. Thank you for your contributions.

@ryalanms ryalanms closed this Jan 12, 2021
@mmitche mmitche deleted the community.testpass.1.4.2021 branch January 13, 2021 14:46
@ghost ghost locked as resolved and limited conversation to collaborators Apr 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
* NO MERGE * metadata: The PR is not ready for merge yet (see discussion for detailed reasons) PR metadata: Label to tag PRs, to facilitate with triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants