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

chore: upgrade deps to prep for 0.2.0-rc1 release #348

Merged
merged 2 commits into from
Jan 13, 2025

Conversation

kentbull
Copy link
Contributor

This does a few things:

  1. bumps version to 0.3.0 for the next release.
  2. Upgrades to KERI 1.2.2 (only had to change the getFelItemPreIter return tuple)
  3. Adds a smaple KERIA config at scripts/keri/cf/keria.json with sample tock configuration.
  4. Updates all dependencies to the latest versions in PyPi.
  5. Uses specific versions in setup.py so builds are strictly repeatable.
  6. adds a little more project metadata

@kentbull kentbull requested a review from m00sey January 10, 2025 19:04
Copy link

codecov bot commented Jan 10, 2025

Codecov Report

Attention: Patch coverage is 28.57143% with 5 lines in your changes missing coverage. Please review.

Project coverage is 93.71%. Comparing base (fa3cb0f) to head (ab20766).
Report is 8 commits behind head on main.

Files with missing lines Patch % Lines
setup.py 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #348      +/-   ##
==========================================
- Coverage   93.77%   93.71%   -0.06%     
==========================================
  Files          37       37              
  Lines        8288     8293       +5     
==========================================
  Hits         7772     7772              
- Misses        516      521       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@iFergal
Copy link
Collaborator

iFergal commented Jan 10, 2025

@kentbull OK, this will supersede #345 - thanks for the extra changes.

However, we haven't released 0.2.0 yet - we're still on dev tags. Hence, I think this PR should upgrade to 0.2.0-rc1 and create a release branch. And then a separate PR can upgrade to 0.3.0-dev0.

scripts/keri/cf/keria.json Outdated Show resolved Hide resolved
setup.py Outdated Show resolved Hide resolved
setup.py Outdated Show resolved Hide resolved
setup.py Outdated Show resolved Hide resolved
@kentbull
Copy link
Contributor Author

kentbull commented Jan 10, 2025

@iFergal sounds good to me, it's now changed to 0.2.0-rc1. I also updated versions in the build pipeline and added the most recent Alpine image.

To slightly optimize image size I swapped out alpine-sdk for gcc. Only the gcc dependency is needed in order to properly link libsodium. If it weren't for linking libsodium then we wouldn't need gcc, which was the entire reason alpine-sdk was included.

@kentbull kentbull changed the title chore: upgrade deps to prep for 0.3.0 release chore: upgrade deps to prep for 0.2.0-rc1 release Jan 10, 2025
@kentbull kentbull requested review from iFergal and 2byrds and removed request for m00sey January 10, 2025 23:21
Copy link
Collaborator

@iFergal iFergal left a comment

Choose a reason for hiding this comment

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

LGTM. Not sure if you're still going to update any docs on the env vs config file but not critical.

Copy link
Collaborator

@2byrds 2byrds left a comment

Choose a reason for hiding this comment

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

LGTM

@kentbull
Copy link
Contributor Author

Yes, I'll update the docs. Looks like I need to fix merge conflicts anyway.

Thanks for the approval @2byrds

@kentbull
Copy link
Contributor Author

@iFergal see the docs I added in the PR. You can cd docs/ && make html && make open to view the docs. I changed the theme to the ReadTheDocs theme which is nice on the eyes.

@kentbull kentbull requested review from iFergal and 2byrds January 13, 2025 17:12
Copy link
Collaborator

@iFergal iFergal left a comment

Choose a reason for hiding this comment

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

LGTM!

@kentbull kentbull merged commit e0d6cb5 into WebOfTrust:main Jan 13, 2025
5 checks passed
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.

3 participants