Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

a-givertzman/va-sensor-emulator

pivovarvalera pushed 1 commit to 16-back-|-va-sensor-|-emulator-|-udpmessage a-givertzman/va-sensor-emulator
  • VA-Sensor | Emulator | UdpMessage 40732ce

View on GitHub

pivovarvalera pushed 1 commit to 15-back-|-va-sensor-|-emulator-|-udpheader a-givertzman/va-sensor-emulator
  • VA-Sensor | Emulator | UdpHeader 1a77d3f

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
replace literal with SYN = 4, defined in the responsible class

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
match send result, just log if error for now

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
Replace literals with constants, defined in the responsible classes if possible.

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
Move to the `udp_header.rs` file - develop in the Issie-PR pair - may bee unit test ?

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
avoid empty lines in the code block

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
Move to the `udp_message.rs` file - develop in the Issie-PR pair - may bee unit test ?

View on GitHub

a-givertzman created a review on a pull request on a-givertzman/va-sensor-emulator

View on GitHub

a-givertzman pushed 2 commits to 13-back-|-va-sensor-|-emulator-|-mainservice a-givertzman/va-sensor-emulator
  • MainService | doc comments fixed b2d2887
  • MainService | doc comments fixed a5f531c

View on GitHub

pivovarvalera pushed 1 commit to 13-back-|-va-sensor-|-emulator-|-mainservice a-givertzman/va-sensor-emulator
  • VA-Sensor | Emulator | MainService 5eb47f8

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
try to avoid unsafe

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
I thought `src/buffer.rs` - was implemented exactly for accumulating with specified capacity like: ```rust match buf.add(amplitude) { Some() => { ... } None -> { ... ...

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
What does it mean? We are already inside `MainService` which is `&mut self` isn't it?

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
↑ prefer in style convenient // //

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
Doc comment required

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
Doc comment required

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
'message()' - can be used

View on GitHub

a-givertzman created a review comment on a pull request on a-givertzman/va-sensor-emulator
empty lines is not required

View on GitHub

Load more