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: add Hashicorp Vault test container and test #148

Merged
merged 4 commits into from
Jun 16, 2024

Conversation

jasondemorrow
Copy link
Contributor

also adds tracing library to dev deps b/c I wanted to see the TC logs and figured others might too

@jasondemorrow jasondemorrow changed the title add Hashicorp Vault test container and test feat: add Hashicorp Vault test container and test Jun 16, 2024
@jasondemorrow jasondemorrow marked this pull request as draft June 16, 2024 03:55
@jasondemorrow
Copy link
Contributor Author

Tests in untouched code are failing, perhaps due to added tracing dep? That doesn't make sense, but it's all I can think of.

@jasondemorrow
Copy link
Contributor Author

Overrode my local 1.79 with 1.75 to match CI, but now aws-sqs won't build b/c it needs 1.76+?

@jasondemorrow
Copy link
Contributor Author

I see what happened here. I had the distinct misfortune of issuing a PR just as the upgrade to TC 0.18 landed in main 😂

@jasondemorrow jasondemorrow marked this pull request as ready for review June 16, 2024 05:20
Copy link
Contributor

@DDtKey DDtKey left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution 🚀

And sorry for this inconvenience after testcontainers update.

@DDtKey DDtKey merged commit f70fb96 into testcontainers:main Jun 16, 2024
6 checks passed
@github-actions github-actions bot mentioned this pull request Jun 16, 2024
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.

None yet

2 participants