8000 Replace JSON examples with YAML && use them in docs by kvch · Pull Request #772 · xataio/pgroll · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Replace JSON examples with YAML && use them in docs #772

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 8000 to your account

Merged
merged 2 commits into from
Apr 2, 2025

Conversation

kvch
Copy link
Contributor
@kvch kvch commented Apr 1, 2025

This PR converts JSON migrations files in examples folder to YAML files. The references to the JSON files in the documentation are replaced with references to YAML migration files.

Requires: #771

@github-actions github-actions bot temporarily deployed to Docs Preview April 1, 2025 11:38 Inactive
@github-actions github-actions bot temporarily deployed to Docs Preview April 1, 2025 11:40 Inactive
@exekias
Copy link
Member
exekias commented Apr 1, 2025

@kvch I thought we wanted to support YAML as a second format, but not drop JSON support. Do we want to switch all examples to yaml?

@kvch
Copy link
Contributor Author
kvch commented Apr 1, 2025

We are not dropping support for JSON. We agreed to change all examples to YAML, and display both YAML and JSON format in the documentations. But we do not need to keep both formats, because we can convert YAML to JSON in the docs automatically.

@kvch kvch force-pushed the replace-json-examples-with-yaml branch from d453bbe to 2e422a3 Compare April 1, 2025 16:15
@github-actions github-actions bot temporarily deployed to Docs Preview April 1, 2025 16:15 Inactive
@kvch kvch merged commit d42d1f8 into xataio:main Apr 2, 2025
30 checks passed
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
0