Skip to content

feat: create Workspace Manager #235

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 14 commits into from
Mar 12, 2025
Merged

feat: create Workspace Manager #235

merged 14 commits into from
Mar 12, 2025

Conversation

seqre
Copy link
Member

@seqre seqre commented Mar 7, 2025

No description provided.

@seqre seqre self-assigned this Mar 7, 2025
Copy link

codecov bot commented Mar 7, 2025

Codecov Report

Attention: Patch coverage is 91.50943% with 9 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cot-cli/src/utils.rs 92.38% 6 Missing and 2 partials ⚠️
cot-cli/src/migration_generator.rs 0.00% 1 Missing ⚠️
Flag Coverage Δ
rust 78.38% <91.50%> (+0.20%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
cot-cli/src/migration_generator.rs 82.64% <0.00%> (ø)
cot-cli/src/utils.rs 91.60% <92.38%> (+30.38%) ⬆️
🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@seqre seqre marked this pull request as ready for review March 9, 2025 13:12
@seqre seqre changed the title Workspace Manager feat: create Workspace Manager Mar 9, 2025
@seqre seqre force-pushed the workspace-mnagement branch from 3a0ffda to df8cbcc Compare March 12, 2025 14:51
@github-actions github-actions bot added the C-cli Crate: cot-cli (issues and Pull Requests related to Cot CLI) label Mar 12, 2025
@seqre seqre requested a review from m4tx March 12, 2025 21:05
Copy link
Member

@m4tx m4tx left a comment

Choose a reason for hiding this comment

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

Please fix the minor issues mentioned and I think it's otherwise a very nice change!

@seqre seqre force-pushed the workspace-mnagement branch from 42d693d to 4792c10 Compare March 12, 2025 22:29
@seqre seqre requested a review from m4tx March 12, 2025 22:29
Copy link
Member

@m4tx m4tx left a comment

Choose a reason for hiding this comment

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

LGTM

@seqre seqre merged commit ca74f41 into master Mar 12, 2025
28 of 29 checks passed
@seqre seqre deleted the workspace-mnagement branch March 12, 2025 22:56
@cotbot cotbot bot mentioned this pull request Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-cli Crate: cot-cli (issues and Pull Requests related to Cot CLI)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants