Skip to content
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

BorshField annotation and enum deser by ordinal #9

Merged
merged 6 commits into from
Mar 2, 2022

Conversation

oak
Copy link
Contributor

@oak oak commented Mar 2, 2022

Implements issues #7 and #8

@oak oak requested a review from AB3rtz March 2, 2022 13:27
@oak oak assigned AB3rtz and oak Mar 2, 2022
@oak oak added the enhancement New feature or request label Mar 2, 2022
@oak oak closed this Mar 2, 2022
@oak oak reopened this Mar 2, 2022
# Conflicts:
#	src/main/java/com/syntifi/near/borshj/BorshInput.java
#	src/main/java/com/syntifi/near/borshj/BorshOutput.java
#	src/main/java/com/syntifi/near/borshj/annotation/BorshField.java
#	src/main/java/com/syntifi/near/borshj/comparator/FieldComparator.java
#	src/main/java/com/syntifi/near/borshj/util/BorshUtil.java
#	src/test/java/com/syntifi/near/borshj/BorshBufferTests.java
#	src/test/java/com/syntifi/near/borshj/FuzzTests.java
#	src/test/java/com/syntifi/near/borshj/pojo/BorshWithMap.java
#	src/test/java/com/syntifi/near/borshj/pojo/BorshWithOptional.java
#	src/test/java/com/syntifi/near/borshj/pojo/BorshWithShape.java
#	src/test/java/com/syntifi/near/borshj/pojo/Circle.java
#	src/test/java/com/syntifi/near/borshj/pojo/PlayerAccount.java
#	src/test/java/com/syntifi/near/borshj/pojo/Point2Df.java
#	src/test/java/com/syntifi/near/borshj/pojo/Rect2Df.java
@oak oak marked this pull request as ready for review March 2, 2022 13:35
@AB3rtz AB3rtz merged commit 67599ed into master Mar 2, 2022
@AB3rtz AB3rtz deleted the features-enum-borshfield branch March 2, 2022 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants