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

dev: Add ddsignfiles wrapper #1774

Closed
wants to merge 2 commits into from
Closed

dev: Add ddsignfiles wrapper #1774

wants to merge 2 commits into from

Conversation

alexweininger
Copy link
Member

This file is a wrapper around the DDSignFiles.dll tool that is present in pipelines that have signing setup and enabled. It has been copied from source because it's easier than consuming an npm package. We can use this js helper to sign files with a js script in each extension instead of requiring a .csproj and building it with MSBuild. I think I'd rather stay in the nodejs world if we can.

@alexweininger alexweininger requested a review from a team as a code owner August 23, 2024 15:20
@alexweininger alexweininger changed the title Add ddsignfiles wrapper dev: Add ddsignfiles wrapper Aug 23, 2024
@bwateratmsft
Copy link
Contributor

Docker and several others do not use the dev package. Can we put this in the pipelines instead?

@alexweininger
Copy link
Member Author

Docker and several others do not use the dev package. Can we put this in the pipelines instead?

Rats. The pipelines folder isn't currently published as a package. Do you know if I include this script in the pipelines folder if it will be accessible within a pipeline run that depends on the templates? I'll try it out.

@alexweininger
Copy link
Member Author

I'm starting to think it might just be easier to use MSBuild and write a csproj file into each repo 😢. Let me try that out too.

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