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

Eliminated warnings and streamlined build #153

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

tombogle
Copy link
Collaborator

@tombogle tombogle commented Feb 6, 2025

This is an attempt to eliminate this warning:
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\Microsoft.Common.CurrentVersion.targets(2401,5): warning MSB3245: Could not resolve this reference. Could not locate the assembly "System.Security.Cryptography.ProtectedData, Version=7.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a, processorArchitecture=MSIL". Check to make sure the assembly exists on disk. If this reference is required by your code, you may get compilation errors.


This change is Reviewable

@tombogle tombogle self-assigned this Feb 6, 2025
…in msbuild)

Upgraded to latest Markdig.Signed
Added explicit references to System.Security to eliminate build warnings
Removed reference to System.Security.Cryptography.ProtectedData from TransceleratorTests.csproj to eliminate build warning
Streamlined some things in the MSBuild proj file
@tombogle tombogle changed the title Add system security cryptography protecteddata Eliminated warnings and streamlined build Feb 7, 2025
@tombogle tombogle added the dependencies Pull requests that update a dependency file label Feb 7, 2025
@tombogle tombogle marked this pull request as ready for review February 7, 2025 19:11
@tombogle tombogle requested a review from andrew-polk February 7, 2025 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant