Skip to content

Conversation

@Nefarious46
Copy link
Contributor

  • Delete mapper returns void
  • Some tests still rely on older version which has not been refactored in this branch.

Pull request #155 has more in-detail comments about changes.

@Nefarious46 Nefarious46 requested a review from kortemik May 14, 2025 10:41
@Nefarious46 Nefarious46 self-assigned this May 14, 2025
@Nefarious46 Nefarious46 added the enhancement New feature or request label May 14, 2025
@Nefarious46 Nefarious46 changed the title Flow endpoint refactor Flow endpoint refactored separately May 14, 2025
public interface FlowMapper {
Flow addNewFlow(String name);
Flow create(String name);

Copy link
Member

Choose a reason for hiding this comment

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

get(Long id) be missing

@kortemik kortemik merged commit 4c88470 into teragrep:main May 19, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants