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

feat(azure-signing-manager): add azure signing manager #52

Closed
wants to merge 2 commits into from

Conversation

polymath-eric
Copy link
Collaborator

Description

add azure signing manager that allows for keys to be stored in an Azure Key Vault and to be used with the Polymesh SDK

Breaking Changes

JIRA Link

DA-1322

Checklist

  • Updated the Readme.md (if required) ?

@polymath-eric polymath-eric force-pushed the azure branch 2 times, most recently from 9e4212d to e8946cc Compare October 4, 2024 21:31
@polymath-eric polymath-eric marked this pull request as ready for review October 4, 2024 21:31
@polymath-eric polymath-eric force-pushed the azure branch 2 times, most recently from e959818 to 3d8367b Compare October 4, 2024 21:35
add azure signing manager that allows for keys to be stored in an Azure Key Vault and to be used
with the Polymesh SDK

DA-1322
@polymath-eric
Copy link
Collaborator Author

Looks like the package "@ng-easy/builders" isn't in npm or yarn repo anymore. Once I deleted node_modules locally yarn broke for me locally with the same error. I thought npm was supposed to keep packages being used, so I am not sure what happened upstream.

Google indexed these links, but looks like its removed from Github and NPM:

Its only used to trigger the semantic release process. Seems like its best to replace it:

@polymath-eric polymath-eric force-pushed the azure branch 4 times, most recently from af9db9b to 2939abd Compare October 7, 2024 17:28
replace @ng-easy/builders with @theunderscorer/nx-semantic-release. @ng-easy seems to have been
removed from npm and github
Copy link

sonarqubecloud bot commented Oct 7, 2024

@polymath-eric
Copy link
Collaborator Author

#53

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.

1 participant