Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

platformutils: drop unnecessary netfx #ifdefs #1447

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

mjcheetham
Copy link
Collaborator

Since we're targetting .NET Framework 4.7.2 there are some unnecessary conditional compliation #if NETFRAMEWORK blocks. Namely around CLR and runtime/OS detection.

Since we're targetting .NET Framework 4.7.2 there are some unnecessary
conditional compliation `#if NETFRAMEWORK` blocks. Namely around CLR and
runtime/OS detection.
@mjcheetham mjcheetham added platform:windows Specific to the Windows platform engineering Refactoring or build changes labels Oct 25, 2023
@mjcheetham mjcheetham enabled auto-merge October 25, 2023 19:12
Copy link
Contributor

@ldennington ldennington left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lovely cleanup!

@mjcheetham mjcheetham merged commit 59f01d9 into git-ecosystem:main Oct 25, 2023
@mjcheetham mjcheetham deleted the netfxcleanup branch October 25, 2023 20:40
mjcheetham added a commit that referenced this pull request Nov 1, 2023
**Changes:**

- Add support for managed identity and service principals in Azure Repos
(#1372)
- Support universal Gitea OAuth app configuration (#1442)
- Set default generic OAuth redirect URI value (#1444)
- Drop WPF helpers on Windows (#1417)
- Add software rendering override for Windows (#1445, #1453)
- Recognise GitLab hosts via WWW-Authenticate header (#1428)
- Recognise Bitbucket hosts via WWW-Authenticate header (#1441)
- Support GitHub Gist remote URLs (#1402)
- Update to Avalonia 11.x (#1383)
- Documentation updates (#1416)
- Drop unnecessary .NET Framework-specific code (#1447)
- Updates to release process (#1386, #1381)
- Update code signing certificates (#1431)
mjcheetham added a commit that referenced this pull request Nov 1, 2023
**Changes:**

_Since 2.4.0:_

- Fix macOS ARM64 tarball contents (#1458)

_Since 2.3.x:_

- Add support for managed identity and service principals in Azure Repos
(#1372)
- Support universal Gitea OAuth app configuration (#1442)
- Set default generic OAuth redirect URI value (#1444)
- Drop WPF helpers on Windows (#1417)
- Add software rendering override for Windows (#1445, #1453)
- Recognise GitLab hosts via WWW-Authenticate header (#1428)
- Recognise Bitbucket hosts via WWW-Authenticate header (#1441)
- Support GitHub Gist remote URLs (#1402)
- Update to Avalonia 11.x (#1383)
- Documentation updates (#1416)
- Drop unnecessary .NET Framework-specific code (#1447)
- Updates to release process (#1386, #1381)
- Update code signing certificates (#1431)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
engineering Refactoring or build changes platform:windows Specific to the Windows platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants