Skip to content

Conversation

@venkatsp17
Copy link
Contributor

@venkatsp17 venkatsp17 commented Aug 19, 2025

Issue #21

@ssanumand
Copy link
Member

ssanumand commented Aug 19, 2025

Looks good to me, please add a reviewer to the PR so that they can review and approve the changes. Once approved, you can merge the branch.

@venkatsp17 venkatsp17 self-assigned this Aug 19, 2025
@venkatsp17 venkatsp17 requested a review from sugan0tech August 19, 2025 18:43
@sugan0tech
Copy link
Member

sugan0tech commented Aug 21, 2025

@venkatsp17 as with the liting pipeline, it appears to be few functions are unused? Will that be covered in following commits?

Copy link
Member

@sugan0tech sugan0tech left a comment

Choose a reason for hiding this comment

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

LGTM, @OpenMigrate/core expecting least one of yours review here.

@sugan0tech sugan0tech merged commit a821fda into main Aug 25, 2025
2 checks passed
@sugan0tech sugan0tech deleted the feat/21-app-grouping branch August 25, 2025 11:29
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.

4 participants