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.

Sign up for GitHub

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

Jump to bottom

Amino 2.0 #318

Open
wants to merge 62 commits into
base: master
Choose a base branch
from
Open

Amino 2.0 #318

wants to merge 62 commits into from

Conversation

jaekwon
Copy link
Contributor

@jaekwon jaekwon commented May 17, 2020 •
edited
Loading

This is not backwards compatible.

  • Native Any support
  • No need for local codecs, just use the global one.
  • Package registration.
  • (genproto) .proto schema generation
  • (genproto) go <---> proto3-generated-go translations
  • Fuzz testing with above translations
  • Convenience for generating types.proto and pb/* protoc and pbbindings.go
  • TODO maybe faster encoding/decoding with protoc.

jaekwon added 30 commits May 14, 2020 17:56
@jaekwon
scan a file and print relevant structs
39f3450
@jaekwon
genproto and plan
c20896e
@jaekwon
Update plan
7da0fc7
@jaekwon
update PLAN
a8d0b75
@jaekwon
update formatting
d4ca65e
@jaekwon
Add CodePress
03df5fa
@jaekwon
add comment
732a6f9
@jaekwon
Sample CodePress usage
431e7ab
@jaekwon
update PLAN Press example
f2f1028
@jaekwon
proper package/imports navigation
5ecb629
@jaekwon
...
42ca14e
@jaekwon
don't do static analysis, use reflection and existing amino TypeInfo
31b6b56
@jaekwon
closer
466bc87
@jaekwon
intermediary
6fd261a
@jaekwon
fix P3Doc import test
60f1032
@jaekwon
fix p3message name in schema
47b1451
@jaekwon
wip
104c503
@jaekwon
...
e578d6c
@jaekwon
Use PackageInfo and make everything explicit
09a0d01
@jaekwon
Fix generated import file, add symlink in proto
c1adb0c
@jaekwon
work in progress with Any
f807819
@jaekwon
fixing tests
8698729
@jaekwon
fixing tests...
cd67dd0
@jaekwon
fix tests...
2edd369
@jaekwon
...
0469e03
@jaekwon
...
ab9b53d
@jaekwon
fixing tests
60692c5
@jaekwon
...
7bd6930
@jaekwon
PackageInfo -> Package, fixing tests
b852644
@jaekwon
...
24be797
@jaekwon jaekwon changed the title WIP genproto plan Amino 1.2 Jul 16, 2020
@jaekwon jaekwon changed the title Amino 1.2 Amino 2.0 Jul 16, 2020
@jaekwon
Copy link
Contributor Author

jaekwon commented Jul 16, 2020 •
edited
Loading

  • amino-x (maybe instead of amino 2.0)
  • not meant to be used with current sdk,core, but for classic.
  • owned by AIB, will be maintained.
  • start of ORM for new tooling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jaekwon