forked from openwallet-foundation/acapy
-
Notifications
You must be signed in to change notification settings - Fork 1
[pull] main from openwallet-foundation:main #215
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Instead of caching to the Runner FS and then uploading it, use native
Docker caching to Github Actions
* Run and fix `zizmor` warnings
* Disable persisting git credentials
* Disable caching of Buildx binary
* Replace unnecessary "Gather image info" step with Github's built in
`${{ github.repository_owner }}` function
* Update `docker/build-push-action` from `v5` to `v6`
Signed-off-by: Robbie Blaine <robbieblaine.rb@gmail.com>
Signed-off-by: Robbie Blaine <robbieblaine.rb@gmail.com>
👷 Fix Docker Caching
* Build Standard image on native CPU architectures * `linux/arm64` builds on `ubuntu-24.04-arm` (`arm64`) * `linux/amd64` builds on `ubuntu-24.04` (`x86_64`) * Split build and publish * Build docker images on native runners, don't push, cache to GHA * Combine multi-arch, read from cache, push to GHCR.io * Due to a version mismatch in buildx between `ubuntu-24.04` and `ubuntu-24.04-arm`, have to ensure the latest version of `buildx` is installed * Due to how the platforms are being handled, I've removed the `platforms` input to the workflow Signed-off-by: Robbie Blaine <robbieblaine.rb@gmail.com>
Signed-off-by: Robbie Blaine <robbieblaine.rb@gmail.com>
👷 Split Docker Builds
* 🎨 Add space to log Signed-off-by: ff137 <ff137@proton.me> * 🎨 Remove unnecessary method (only used once) Signed-off-by: ff137 <ff137@proton.me> * 🎨 Remove unused arg Signed-off-by: ff137 <ff137@proton.me> * 🎨 Remove f-string formatting from logs; fix string concatenation Signed-off-by: ff137 <ff137@proton.me> * 🐛 Add missing return statement Signed-off-by: ff137 <ff137@proton.me> * 🎨 More removing f-string format from logs Signed-off-by: ff137 <ff137@proton.me> * 🎨 Clarify log Signed-off-by: ff137 <ff137@proton.me> * 🐛 Fix: return public_did_info. Resolves #3645 Signed-off-by: ff137 <ff137@proton.me> * 🎨 Clean up `_initialize_with_public_did` method. Rename to `_replace_public_did_if_seed_mismatch` for clarity, and simplify logic Signed-off-by: ff137 <ff137@proton.me> * 🎨 None safety Signed-off-by: ff137 <ff137@proton.me> * 🎨 Remove extra space Signed-off-by: ff137 <ff137@proton.me> * 🎨 Remove log method and just call directly Signed-off-by: ff137 <ff137@proton.me> * 🐛 Add missing return statement Signed-off-by: ff137 <ff137@proton.me> * ✅ Expand test coverage for wallet config Signed-off-by: ff137 <ff137@proton.me> --------- Signed-off-by: ff137 <ff137@proton.me>
Signed-off-by: Robbie Blaine <robbieblaine.rb@gmail.com>
* 🎨 Use revocation_anoncreds tag title Signed-off-by: ff137 <ff137@proton.me> * 🎨 Deduplicate tag titles Signed-off-by: ff137 <ff137@proton.me> * 🎨 Deduplicate tag titles Signed-off-by: ff137 <ff137@proton.me> * 🎨 Avoid circular import, due to knot between anoncreds/routes and revocation_anoncreds/routes Signed-off-by: ff137 <ff137@proton.me> * 🎨 Fix new lines in description summary Signed-off-by: ff137 <ff137@proton.me> * 📝 Update openapi spec Signed-off-by: ff137 <ff137@proton.me> * 🎨 Reword update description Signed-off-by: ff137 <ff137@proton.me> --------- Signed-off-by: ff137 <ff137@proton.me> Co-authored-by: Stephen Curran <swcurran@gmail.com>
* ✅ Correctly tear down ClientSession in TestAdminServer Signed-off-by: ff137 <ff137@proton.me> * 🎨 Ignore create_did_document deprecation warning in test run Signed-off-by: ff137 <ff137@proton.me> * 🧪 async test fixtures should use pytest_asyncio.fixture Signed-off-by: ff137 <ff137@proton.me> * 🎨 Add missing pytest.mark.asyncio Signed-off-by: ff137 <ff137@proton.me> * 🎨 Ignore create_did_document deprecation warning in test run Signed-off-by: ff137 <ff137@proton.me> --------- Signed-off-by: ff137 <ff137@proton.me> Co-authored-by: Stephen Curran <swcurran@gmail.com>
* 🎨 Add type hints Signed-off-by: ff137 <ff137@proton.me> * 🎨 Add todo comments. #3651 Signed-off-by: ff137 <ff137@proton.me> * 🎨 Add return types Signed-off-by: ff137 <ff137@proton.me> --------- Signed-off-by: ff137 <ff137@proton.me>
Signed-off-by: ff137 <ff137@proton.me>
Bumps [markdown](https://github.com/Python-Markdown/markdown) from 3.7 to 3.8. - [Release notes](https://github.com/Python-Markdown/markdown/releases) - [Changelog](https://github.com/Python-Markdown/markdown/blob/master/docs/changelog.md) - [Commits](Python-Markdown/markdown@3.7...3.8) --- updated-dependencies: - dependency-name: markdown dependency-version: '3.8' dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: jamshale <31809382+jamshale@users.noreply.github.com>
* 🎨 Make ledger config more readable Signed-off-by: ff137 <ff137@proton.me> * 🎨 Fix `write_ledger_set` checking all configs Signed-off-by: ff137 <ff137@proton.me> * ✅ Fix expected error message `No writable ledger configured` instead of `No is_write ledger set` Signed-off-by: ff137 <ff137@proton.me> * 🎨 Add logging to ledger arg parse Signed-off-by: ff137 <ff137@proton.me> * 🎨 Clarify single_configured, multi_configured Signed-off-by: ff137 <ff137@proton.me> * 🎨 Signed-off-by: ff137 <ff137@proton.me> * 🎨 Reduce duplication and complexity Signed-off-by: ff137 <ff137@proton.me> * 🎨 Add logging to `get_write_ledger_config_for_profile`, and simplify key reading Signed-off-by: ff137 <ff137@proton.me> * 🎨 Signed-off-by: ff137 <ff137@proton.me> * 🎨 Reduce complexity in binding multi-ledger provider Signed-off-by: ff137 <ff137@proton.me> --------- Signed-off-by: ff137 <ff137@proton.me>
* fix for inline VMs Signed-off-by: George Mulhearn <gmulhearn@anonyome.com> * Signed-off-by: George Mulhearn <gmulhearn@anonyome.com> * retrigger CI Signed-off-by: George Mulhearn <gmulhearn@anonyome.com> --------- Signed-off-by: George Mulhearn <gmulhearn@anonyome.com> Co-authored-by: George Mulhearn <gmulhearn@anonyome.com>
* 🎨 Rename did:indy create/response schema objects Signed-off-by: ff137 <ff137@proton.me> * 🎨 Add more detail to options description Signed-off-by: ff137 <ff137@proton.me> * 🎨 Clarify defaults Signed-off-by: ff137 <ff137@proton.me> --------- Signed-off-by: ff137 <ff137@proton.me> Co-authored-by: Stephen Curran <swcurran@gmail.com>
Bumps [packaging](https://github.com/pypa/packaging) from 24.2 to 25.0. - [Release notes](https://github.com/pypa/packaging/releases) - [Changelog](https://github.com/pypa/packaging/blob/main/CHANGELOG.rst) - [Commits](pypa/packaging@24.2...25.0) --- updated-dependencies: - dependency-name: packaging dependency-version: '25.0' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…3668) Signed-off-by: StepSecurity Bot <bot@stepsecurity.io>
Flattens duplication by specifying "directories". Adds `/scenarios` directory to pip and docker management Signed-off-by: ff137 <ff137@proton.me>
Signed-off-by: StepSecurity Bot <bot@stepsecurity.io> Co-authored-by: Stephen Curran <swcurran@gmail.com>
* 🔒 Update Token Permissions in GitHub Actions Signed-off-by: ff137 <ff137@proton.me> * 🔒 Move packages: write permission to job level Signed-off-by: ff137 <ff137@proton.me> --------- Signed-off-by: ff137 <ff137@proton.me>
…24 (#3683) Bumps [pydevd-pycharm](https://github.com/JetBrains/intellij-community) from 251.23774.211 to 251.25410.24. - [Commits](JetBrains/intellij-community@pycharm/251.23774.211...pycharm/251.25410.24) --- updated-dependencies: - dependency-name: pydevd-pycharm dependency-version: 251.25410.24 dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Stephen Curran <swcurran@gmail.com>
Bumps untergeek/curator from 8.0.16 to 8.0.21. --- updated-dependencies: - dependency-name: untergeek/curator dependency-version: 8.0.21 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: jamshale <31809382+jamshale@users.noreply.github.com> Co-authored-by: Stephen Curran <swcurran@gmail.com>
This is because GitHub Actions does not provide repository secrets (such as SONAR_TOKEN) to workflows triggered by dependabot[bot] for security reasons. As a result, the SonarCloud scan step fails when dependabot merges to main (i.e., using `@dependabot squash and merge`, instead of manually merging oneself). By skipping the scan when the actor is dependabot, we avoid unnecessary workflow failures. Signed-off-by: ff137 <ff137@proton.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
fix: multiuse invite derived conns should have msg id
* 🎨 Fix type hints Signed-off-by: ff137 <ff137@proton.me> * 🐛 Handle TAA response missing expected key Signed-off-by: ff137 <ff137@proton.me> --------- Signed-off-by: ff137 <ff137@proton.me> Co-authored-by: Stephen Curran <swcurran@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.1)
Can you help keep this open source service alive? 💖 Please sponsor : )