Skip to content

Commit

Permalink
Remove SponsorLink since it breaks MacOS restore
Browse files Browse the repository at this point in the history 
Fixes #1371
  • Loading branch information
@kzu
kzu committed Aug 9, 2023
1 parent d6d8018 commit a7dcd43
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions src/Moq/Moq.csproj
View file
Expand Up @@ -110,8 +110,4 @@
<InternalsVisibleTo Include="DynamicProxyGenAssembly2" PublicKey="0024000004800000940000000602000000240000525341310004000001000100c547cac37abd99c8db225ef2f6c8a3602f3b3606cc9891605d02baa56104f4cfc0734aa39b93bf7852f7d9266654753cc297e7d2edfe0bac1cdcf9f717241550e0a7b191195b7667bb4f64bcb8e2121380fd1d9d46ad2d92d2d15605093924cceaf74c4861eff62abf69b9291ed0a340e113be11e6a7d3113e92484cf7045cc7" />
</ItemGroup>

<ItemGroup>
<ProjectReference Include="..\Moq.CodeAnalysis\Moq.CodeAnalysis.csproj" ReferenceOutputAssembly="false" />
</ItemGroup>

</Project>

12 comments on commit a7dcd43

@Laiteux
Copy link

@Laiteux Laiteux commented on a7dcd43 Aug 9, 2023

Choose a reason for hiding this comment

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

lol

@Ivandavidmircea
Copy link

Choose a reason for hiding this comment

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

image

@dartasen
Copy link

Choose a reason for hiding this comment

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

That's gross

@mirata
Copy link

@mirata mirata commented on a7dcd43 Aug 10, 2023

Choose a reason for hiding this comment

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

Yep that’s the only reason to remove it

@motmot80
Copy link

Choose a reason for hiding this comment

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

A privacy statement should be added. Companies really don't like it when data is sent out of their build infrastructure.

@HeyBanditoz
Copy link

@HeyBanditoz HeyBanditoz commented on a7dcd43 Aug 10, 2023 •
edited

Choose a reason for hiding this comment

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

Caught in 4k
4k-caught

@Ayymoss
Copy link

Choose a reason for hiding this comment

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

Are you trying to fool the developer community? Good luck restoring your reputation...

@meowzin644
Copy link

Choose a reason for hiding this comment

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

And that's how you kill a project. Trust is completely shattered.

@gnalvesteffer
Copy link

Choose a reason for hiding this comment

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

RIP. Reputation down the shitter.

@jingeno
Copy link

Choose a reason for hiding this comment

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

I'm glad it has been removed but it's very disappointing that it got introduced in the first place

@Zulgrib
Copy link

Choose a reason for hiding this comment

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

https://www.spiritlegal.com/en/news/details/e-commerce-retail-facebook-custom-audience-not-allowed-without-consent.html

SHA 256 isn't a proper method for that

@suntereo
Copy link

Choose a reason for hiding this comment

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

I don't understand the reason for removal. Was it not because of privacy concerns? If it hadn't broken MacOS restore would it still be in the code base? If so, I'm nervous about continued use with this package.

Please sign in to comment.