-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Big Trait Changes #206
Open
TTWNO
wants to merge
43
commits into
main
Choose a base branch
from
checked-and-unchecked-tryfrom-msg
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Big Trait Changes #206
Commits on Jul 5, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 015ccc2 - Browse repository at this point
Copy the full SHA 015ccc2View commit details -
Configuration menu - View commit details
-
Copy full SHA for dd10886 - Browse repository at this point
Copy the full SHA dd10886View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6151aa5 - Browse repository at this point
Copy the full SHA 6151aa5View commit details -
Configuration menu - View commit details
-
Copy full SHA for a3f596f - Browse repository at this point
Copy the full SHA a3f596fView commit details -
Configuration menu - View commit details
-
Copy full SHA for fe46a31 - Browse repository at this point
Copy the full SHA fe46a31View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5ad158f - Browse repository at this point
Copy the full SHA 5ad158fView commit details -
Configuration menu - View commit details
-
Copy full SHA for bc865fe - Browse repository at this point
Copy the full SHA bc865feView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6ad6a16 - Browse repository at this point
Copy the full SHA 6ad6a16View commit details -
Configuration menu - View commit details
-
Copy full SHA for fe9da31 - Browse repository at this point
Copy the full SHA fe9da31View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5add99d - Browse repository at this point
Copy the full SHA 5add99dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8578f85 - Browse repository at this point
Copy the full SHA 8578f85View commit details -
Configuration menu - View commit details
-
Copy full SHA for 1b2a734 - Browse repository at this point
Copy the full SHA 1b2a734View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0aa7380 - Browse repository at this point
Copy the full SHA 0aa7380View commit details -
Configuration menu - View commit details
-
Copy full SHA for e86643c - Browse repository at this point
Copy the full SHA e86643cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5f52f93 - Browse repository at this point
Copy the full SHA 5f52f93View commit details -
Change trait bound to be more generic
Change trait bounds for auto-impl of MessageConversion
Configuration menu - View commit details
-
Copy full SHA for 3368b83 - Browse repository at this point
Copy the full SHA 3368b83View commit details -
Configuration menu - View commit details
-
Copy full SHA for a9bb246 - Browse repository at this point
Copy the full SHA a9bb246View commit details -
Configuration menu - View commit details
-
Copy full SHA for a47aaf4 - Browse repository at this point
Copy the full SHA a47aaf4View commit details -
Configuration menu - View commit details
-
Copy full SHA for c4e0c1e - Browse repository at this point
Copy the full SHA c4e0c1eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1b3791d - Browse repository at this point
Copy the full SHA 1b3791dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 123e47d - Browse repository at this point
Copy the full SHA 123e47dView commit details -
Add new tests around message conversion
- Correct interface, member and invalid body - Correct body, interface and invalid member
Configuration menu - View commit details
-
Copy full SHA for df1e36b - Browse repository at this point
Copy the full SHA df1e36bView commit details -
The order of validation is the following when using `Event::try_from(Message)`: - MissingInterface - InterfaceMatch - MissingMember - MemberMatch - MissingSignature - SignatureMatch Or at least, that's what it should be. THe commit makes individual implementations of `TryFrom<Message>` cohere to the same order of validation.
Configuration menu - View commit details
-
Copy full SHA for e4359f2 - Browse repository at this point
Copy the full SHA e4359f2View commit details -
Configuration menu - View commit details
-
Copy full SHA for e5a0374 - Browse repository at this point
Copy the full SHA e5a0374View commit details -
Configuration menu - View commit details
-
Copy full SHA for c9857a1 - Browse repository at this point
Copy the full SHA c9857a1View commit details -
Configuration menu - View commit details
-
Copy full SHA for ad74e27 - Browse repository at this point
Copy the full SHA ad74e27View commit details -
Configuration menu - View commit details
-
Copy full SHA for 791dad3 - Browse repository at this point
Copy the full SHA 791dad3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5107e2c - Browse repository at this point
Copy the full SHA 5107e2cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2ff735f - Browse repository at this point
Copy the full SHA 2ff735fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9052cf5 - Browse repository at this point
Copy the full SHA 9052cf5View commit details -
Configuration menu - View commit details
-
Copy full SHA for d9204b6 - Browse repository at this point
Copy the full SHA d9204b6View commit details -
Configuration menu - View commit details
-
Copy full SHA for c3bbdd2 - Browse repository at this point
Copy the full SHA c3bbdd2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 1b1f76c - Browse repository at this point
Copy the full SHA 1b1f76cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 370c295 - Browse repository at this point
Copy the full SHA 370c295View commit details -
Configuration menu - View commit details
-
Copy full SHA for 45d6cd0 - Browse repository at this point
Copy the full SHA 45d6cd0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 855d4df - Browse repository at this point
Copy the full SHA 855d4dfView commit details -
Configuration menu - View commit details
-
Copy full SHA for c7a657e - Browse repository at this point
Copy the full SHA c7a657eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 73c17b0 - Browse repository at this point
Copy the full SHA 73c17b0View commit details -
Add new methods on trait
MessageConversion
- `validate_interface` (auto-impl) - `validate_member` (auto-impl) - `validate_body` (auto-impl) - `try_from_validated_message_parts` (required) - This constructs the event from an to it with the arguments ObjectRef,Self::Body
Configuration menu - View commit details
-
Copy full SHA for ba2df3b - Browse repository at this point
Copy the full SHA ba2df3bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 96387e8 - Browse repository at this point
Copy the full SHA 96387e8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 866415b - Browse repository at this point
Copy the full SHA 866415bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 31abd35 - Browse repository at this point
Copy the full SHA 31abd35View commit details -
Configuration menu - View commit details
-
Copy full SHA for cb70e23 - Browse repository at this point
Copy the full SHA cb70e23View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.