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

Removing dead code after TLS activation work #479

Merged

Conversation

darcythomas
Copy link
Contributor

@darcythomas darcythomas commented May 25, 2023

Tidy up release

  1. Remove dead code
  2. Added nuget unit test project
  3. Sign regular packages too
  4. Fix icon metadata
  5. Fix licence metadata

@darcythomas darcythomas changed the title Darcy/mc 89/net framework tls activation sans net2 3 rt Removing dead code after tls activation work May 28, 2023
@darcythomas darcythomas changed the title Removing dead code after tls activation work Removing dead code after TLS activation work May 28, 2023
less ws

Less UWP for win 8

it all builds! Apart from the win store

wip

bump major version

fix

Update Mindscape.Raygun4Net.Common.csproj

Update Mindscape.Raygun4Net.Azure.WebJob.csproj

less .net 3.0

tidy

Remove .Net 2-3.5 and windows phone

Update Mindscape.Raygun4Net.AspNetCore.sln

Update readme.txt

Update README.md

now can package

Better building

Now with moar tls!

typo

Update readme.txt

Update README.md

bump major version + getting psake builds to work correctly

Tidys

Update TestIfWeCanSendToRaygun.cs

Removed the tests which proved what the behavior was before we did these changes

Tidy

Fix for broken tests

Now even better!

getting close

Update WebClientHelper.cs

Extracting interface

Ok it builds

porting moar tests

Tidy

wip

clean up

new integration tests

Basic  integrations tests setup

Now with green tests!

Got tests properly done

partial fix

fix to tests to validate short comings of old code

tidy

wip tidy up

Update UnitTest1.cs

tidy

Update TestIfWeCanSendToRaygun.cs

Update UnitTest1.cs

Update UnitTest1.cs

Update RaygunClient.cs
@darcythomas darcythomas force-pushed the darcy/MC-89/NET-Framework-TLS-activation-sans-net2-3-RT branch from 07433cf to 10ffb3d Compare June 7, 2023 00:25
Copy link
Contributor

@xenolightning xenolightning left a comment

Choose a reason for hiding this comment

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

Excellent clean up🤘🏼

Remove the old unsupported platforms and 3.5 code and targeting, and updating docs accordingly

@darcythomas darcythomas merged commit 65b8778 into master Jun 8, 2023
@darcythomas darcythomas deleted the darcy/MC-89/NET-Framework-TLS-activation-sans-net2-3-RT branch June 8, 2023 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants