Remove reference and use of dataStream everywhere in tests
thehenrytsai opened this issue · comments
Henry Tsai commented
Background:
I believe existing ProtocolsConfigure
needlessly passes an empty data stream to processMessage
, it is unnecessary because we made a decision (#139) to have the protocol definition defined in the descriptor
.
Task Details:
- Remove reference and use of dataStream everywhere in tests
Picking Up This Issue:
- If you'd like to work on this, please comment "picking this up" below, and I'll assign the issue to you
Questions:
- If you need assistance or clarification, feel free to comment below.
- Feeling overwhelmed? Join our
#hack-together
channel and collaborate with a buddy. - New to open-source? Check out our Hacktober
#getting-started
channel. - Leaderboard: Check your ranking after finishing this task.
Resources:
- Creating a Pull Request: If you're new to GitHub and unsure how to create a pull request, follow this step-by-step guide.
Remember, communication is key! If you have any questions or face any challenges, we're here to help so please don't hesitate to reach out.
Good Luck! 🍁
Josh commented
Picking this up