Add ACK and NAK PDU abstractions #41

Merged

muellerr merged 19 commits from add-ack-nak-pdus into main 2023-12-01 15:33:36 +01:00

Conversation 0 Commits 19 Files Changed 10

+1324 -36

muellerr commented 2023-11-27 15:50:42 +01:00

Owner

Copy Link

Copy Source

No description provided.

muellerr added 2 commits 2023-11-27 15:50:43 +01:00

added basic state tests for FD and Finished PDU 9976b53f65

added ACK PDU impl fdf6e1de90

muellerr added 2 commits 2023-11-27 15:52:57 +01:00

first test c0805db137

Serialize/Deserialize c0bedac058

muellerr added 1 commit 2023-11-27 16:26:03 +01:00

added some tests for new ACK PDU dcb697bf6f

muellerr added 1 commit 2023-11-27 17:48:12 +01:00

added NAK PDU impl 65d3b4e4e5

muellerr added 1 commit 2023-11-28 10:04:13 +01:00

split up NAK PDU API 6ed023d50d

muellerr added 1 commit 2023-11-28 22:29:19 +01:00

add test b8dacc12b5

muellerr added 1 commit 2023-11-28 22:33:56 +01:00

last push 5ae86619b4

muellerr added 1 commit 2023-11-30 20:20:37 +01:00

that was complicated but it works 80aa963226

muellerr added 1 commit 2023-11-30 23:00:04 +01:00

add more tests 2ba2998426

muellerr added 1 commit 2023-12-01 10:22:50 +01:00

added coverage helper script c99b6fddec

muellerr added 1 commit 2023-12-01 10:33:19 +01:00

added coverage section in README 94ff4fbb51

muellerr added 1 commit 2023-12-01 12:11:36 +01:00

improve coverage ea1edfb3c1

muellerr added 1 commit 2023-12-01 14:18:41 +01:00

NAK PDU crc check 56113ffbcb

muellerr added 1 commit 2023-12-01 14:22:41 +01:00

fix for serde e422f4f969

muellerr added 1 commit 2023-12-01 14:27:27 +01:00

serde impl is tricky here.. e2ae959d03

muellerr marked the pull request as ready for review 2023-12-01 14:30:12 +01:00

muellerr added 1 commit 2023-12-01 14:34:21 +01:00

remove std usages 1f1aa68485

muellerr added 1 commit 2023-12-01 14:46:10 +01:00

fmt e355de3f10

muellerr merged commit 80b80f6777 into main 2023-12-01 15:33:36 +01:00

muellerr referenced this issue from a commit 2023-12-01 15:33:36 +01:00

Merge pull request 'Add ACK and NAK PDU abstractions' (#41) from add-ack-nak-pdus into main

muellerr deleted branch add-ack-nak-pdus 2023-12-01 15:33:37 +01:00

muellerr added this to the v0.7.0 milestone 2023-12-01 15:34:50 +01:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

api

Non-Breaking API change

b-api

Breaking API change

bug

Something is not working

concepts

Conception and ideas for new features

duplicate

This issue or pull request already exists

enhancement

New feature

help wanted

Need some help

invalid

Something is wrong

question

More information is needed

wontfix

This won't be fixed

No Label

Milestone

No items

No Milestone v0.7.0

Projects

Clear projects

No project

Assignees

Clear assignees

ahinkel (Andy Hinkel) jenkins_buildfix lkoester meggert (Marius Eggert) meierj (Jakob Meier) mhess mohr (Ulrich Mohr) muellerr (Robin Müller) nehlichp (Paul Nehlich) steinertm (Michael Steinert)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: rust/spacepackets#41