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

feat: api gen and docsgen #45

Merged
merged 22 commits into from
Jun 11, 2024
Merged

feat: api gen and docsgen #45

merged 22 commits into from
Jun 11, 2024

Conversation

snadrus
Copy link
Contributor

@snadrus snadrus commented Jun 9, 2024

Make gen
Command fully implemented. No types (however, go generate added), because types are used from Lotus.

Tests are included to ensure that the work is done.

@snadrus snadrus requested review from magik6k and LexLuthr June 9, 2024 03:04
Copy link
Contributor

@LexLuthr LexLuthr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The "documentation/en" is reserved for our main docs.

documentation/en/api-v0-methods-curio.md Outdated Show resolved Hide resolved
@LexLuthr LexLuthr changed the title Feat/docs and api feat: api gen and docsgen Jun 10, 2024
Copy link
Contributor

@LexLuthr LexLuthr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few small changes required

api/docgen-openrpc/cmd/docgen_openrpc.go Outdated Show resolved Hide resolved
api/docgen-openrpc/cmd/docgen_openrpc.go Outdated Show resolved Hide resolved
api/gen/bundle/bundle.go Outdated Show resolved Hide resolved
api/gen/main.go Outdated Show resolved Hide resolved
api/docgen-openrpc/openrpc.go Outdated Show resolved Hide resolved
api/docgen-openrpc/openrpc.go Outdated Show resolved Hide resolved
api/docgen-openrpc/openrpc.go Outdated Show resolved Hide resolved
api/gen/api/proxygen.go Outdated Show resolved Hide resolved
build/openrpc/curio.json Outdated Show resolved Hide resolved
Copy link
Contributor

@LexLuthr LexLuthr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@LexLuthr LexLuthr merged commit 6fad3fa into main Jun 11, 2024
7 checks passed
@LexLuthr LexLuthr deleted the feat/docsAndApi branch June 11, 2024 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants