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: Generate docs from JSON schema #50

Merged
merged 31 commits into from
Oct 17, 2023
Merged

feat: Generate docs from JSON schema #50

merged 31 commits into from
Oct 17, 2023

Conversation

candiduslynx
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the feat label Oct 12, 2023
@candiduslynx candiduslynx marked this pull request as ready for review October 12, 2023 09:40
Copy link
Member

@hermanschaaf hermanschaaf left a comment

Choose a reason for hiding this comment

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

Very cool, and I think we can merge this in as a starting point. Before we can use it though we'll need to incorporate descriptions for the fields as well. In this package it seems relatively straight-forward, we can use the description field if set.

For generating the schemas from Go code though, maybe we can read the comments above each field using reflection, like we used to do in CQ v0 for generating table column descriptions.

jsonschema/docs/docs_test.go Outdated Show resolved Hide resolved
@candiduslynx candiduslynx added the automerge Add to automerge PRs once requirements are met label Oct 17, 2023
@kodiakhq kodiakhq bot merged commit ce84add into main Oct 17, 2023
9 checks passed
@kodiakhq kodiakhq bot deleted the feat/jsonschema/doc branch October 17, 2023 16:28
kodiakhq bot pushed a commit that referenced this pull request Oct 18, 2023
🤖 I have created a release *beep* *boop*
---


## [0.3.9](v0.3.8...v0.3.9) (2023-10-18)


### Features

* Add options appicable to `jsonschema.Reflector` prior to generation ([#60](#60)) ([2cace1e](2cace1e))
* Generate docs from JSON schema ([#50](#50)) ([ce84add](ce84add))


### Bug Fixes

* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.13.0 ([#55](#55)) ([507131a](507131a))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.14.0 ([#57](#57)) ([31ade81](31ade81))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.14.1 ([#58](#58)) ([6002349](6002349))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.15.0 ([#59](#59)) ([2c6f5a8](2c6f5a8))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.15.1 ([#61](#61)) ([dca6592](dca6592))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Add to automerge PRs once requirements are met feat
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants