Skip to content

Conversation

egregius313
Copy link
Contributor

No description provided.

@egregius313 egregius313 added the no-change-note-required This PR does not need a change note label Apr 3, 2023
@egregius313 egregius313 requested a review from a team as a code owner April 3, 2023 18:03
@github-actions github-actions bot added the Java label Apr 3, 2023
Copy link
Contributor

@atorralba atorralba left a comment

Choose a reason for hiding this comment

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

Mostly LGTM, with a couple of comments.

@egregius313 egregius313 force-pushed the egregius313/dataflow-refactor-cleanup branch from 889a974 to 72907eb Compare April 10, 2023 15:41
@egregius313 egregius313 requested a review from atorralba April 11, 2023 16:08
Copy link
Contributor

@atorralba atorralba left a comment

Choose a reason for hiding this comment

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

Just a couple more minor things.

@egregius313 egregius313 force-pushed the egregius313/dataflow-refactor-cleanup branch from 9c80814 to 0a26916 Compare April 14, 2023 03:06
@egregius313 egregius313 requested a review from atorralba April 14, 2023 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Java no-change-note-required This PR does not need a change note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants