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

7640 docker test vaults #7717

Merged
merged 8 commits into from
May 13, 2023
Merged

7640 docker test vaults #7717

merged 8 commits into from
May 13, 2023

Conversation

turadg
Copy link
Member

@turadg turadg commented May 12, 2023

closes: #7640
refs: #XXXX

Description

draft with @raphdev

Security Considerations

Scaling Considerations

Documentation Considerations

Testing Considerations

@turadg turadg mentioned this pull request May 12, 2023
@arirubinstein arirubinstein force-pushed the 7640-docker-test-vaults branch 2 times, most recently from 81830fd to 2564291 Compare May 13, 2023 01:03
@arirubinstein arirubinstein marked this pull request as ready for review May 13, 2023 01:05
@turadg turadg marked this pull request as draft May 13, 2023 14:55
@turadg turadg marked this pull request as ready for review May 13, 2023 18:26
@turadg turadg changed the base branch from master to ta/cli-fixes May 13, 2023 18:29
Copy link
Contributor

@raphdev raphdev left a comment

Choose a reason for hiding this comment

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

Approving as I have tested many revisions of this PR, including a similar set of squashed/fixup commits at raph/7640-docker-test-vaults-fixup based on earlier versions of this branch.

The latest rebase worked fine for me as well doing a clean make local_sdk && make build.

Base automatically changed from ta/cli-fixes to master May 13, 2023 18:54
@raphdev raphdev added this pull request to the merge queue May 13, 2023
Merged via the queue into master with commit d06717b May 13, 2023
@raphdev raphdev deleted the 7640-docker-test-vaults branch May 13, 2023 19:25
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.

automated test coverage of functionality after Pismo -> Vaults upgrade
3 participants