Skip to content

Comments

fix: removes deprecated docker compose field#647

Merged
tdstein merged 1 commit intomainfrom
tdstein/644
Mar 19, 2025
Merged

fix: removes deprecated docker compose field#647
tdstein merged 1 commit intomainfrom
tdstein/644

Conversation

@tdstein
Copy link
Collaborator

@tdstein tdstein commented Mar 19, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Mar 19, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
4868 3621 74% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 947ee07 by action🐍

@tdstein tdstein changed the title fix: integration tests on connect fix: removes deprecated docker compose field Mar 19, 2025
@tdstein tdstein marked this pull request as ready for review March 19, 2025 17:32
@tdstein tdstein requested a review from jonkeane March 19, 2025 17:32
@tdstein
Copy link
Collaborator Author

tdstein commented Mar 19, 2025

@jonkeane - this gets rid of the warning message but doesn't fix CI. That's a bigger fix, which I'll do in a separate PR.

Copy link
Collaborator

@jonkeane jonkeane left a comment

Choose a reason for hiding this comment

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

Thanks for this. I'll leave it up to you if you want to merge this or just leave if off and deal with fixing the CI later — either is totally fine.

@tdstein tdstein merged commit 9f7c114 into main Mar 19, 2025
11 of 13 checks passed
@tdstein tdstein deleted the tdstein/644 branch March 19, 2025 19:27
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.

2 participants