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.

Sign up for GitHub

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

Jump to bottom

Refactor to use new crypto-ld and jsigs api. #50

Merged
merged 22 commits into from
Mar 19, 2021
Merged

Refactor to use new crypto-ld and jsigs api. #50

merged 22 commits into from
Mar 19, 2021

Conversation

dmitrizagidulin
Copy link
Contributor

@dmitrizagidulin dmitrizagidulin commented Aug 10, 2020 •
edited
Loading

@dmitrizagidulin dmitrizagidulin mentioned this pull request Aug 10, 2020
Merged
@davidlehn
Fix coverage output.
ee4b53d
@davidlehn
Update dependencies.
4472105
@davidlehn
Update workflow.
a5b4f80 
- Remove CI env.
- Use and test with Node.js 14.x.
- Add timeouts.
@davidlehn
Update API usage.
8365769
@davidlehn
Update changelog.
347516c
@davidlehn
Only support Node.js >=12.
f3715f4
@davidlehn
Remove unused plugins config.
0c29014
@davidlehn
Update dependencies.
09e2b4a 
- Use modern tools.
- Ship bundled ESM version.
@davidlehn
Rename package.
a3ae7af 
- Changed package name from `ocapld` to `@digitalbazaar/zcapld`.
@dmitrizagidulin dmitrizagidulin mentioned this pull request Mar 18, 2021
Merged
4 tasks
dlongley
dlongley approved these changes Mar 19, 2021
View reviewed changes
Copy link
Member

@dlongley dlongley left a comment

Choose a reason for hiding this comment

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

Rubberstamping... seems like we've covered most (all?) of the things we wanted to do with this release.

@mattcollier
Copy link
Contributor

@dmitrizagidulin CI is not passing (karma test failing). Is that a known issue?

mattcollier
mattcollier approved these changes Mar 19, 2021
View reviewed changes
Copy link
Contributor

@mattcollier mattcollier left a comment

Choose a reason for hiding this comment

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

LGTM once CI is passing.

@davidlehn
Update keywords.
e0a7c7f
@davidlehn
Prepare verions for release.
7f107e6 
- Update from ocapld version to prepare for new zcpald 1.0.0.
@dmitrizagidulin dmitrizagidulin merged commit 71f944a into master Mar 19, 2021
@mattcollier mattcollier deleted the v3.x branch April 15, 2021 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mattcollier mattcollier mattcollier approved these changes

@dlongley dlongley dlongley approved these changes

@davidlehn davidlehn davidlehn approved these changes

@gannan08 gannan08 Awaiting requested review from gannan08

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dmitrizagidulin @mattcollier @davidlehn @dlongley