Skip to content

Conversation

@royendo
Copy link
Contributor

@royendo royendo commented Aug 4, 2025

Separated the schema files for easier management
referenced in project.schema.yaml.

reworked connections to look similar to currrent with copiable form.
models.schema.yaml --> advanced-model.schema.yaml

should we move model.sql reference into the ref folde rmanually?

once confirmed, we can move this to rewrite /reference/project--files/

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

unsur eif schema./yaml is maintained or auto generated.
@royendo royendo changed the title Update the auto Update the auto YAML Generation Aug 4, 2025
@royendo royendo marked this pull request as ready for review August 4, 2025 21:18
@royendo royendo requested review from NamanMahor and mindspank August 4, 2025 21:18
@royendo
Copy link
Contributor Author

royendo commented Aug 5, 2025

@mindspank thoughts on this PR?

  • changed hidden names to match current files
  • separated the schemas into different files
  • made some changes to generate code for connector to make it similar to current look, (copiable code blocks)

If approved, ill go back in and overwrite /reference/project-files/ and fix any broken links

@NamanMahor
Copy link
Contributor

@royendo , we prefer using a single file instead of using imports — like in this example, where everything is in one file. Initially, I had created separate files and imported them into project.schema.yaml, like you did, but @begelundmuller suggested keeping everything in one file.

royendo added 5 commits August 6, 2025 09:03
all schemas into one including rillyaml. commented out lines.

reworked generate-project.go:
- moved out logic for sample in copy, uses schema sample instead
- adding in line examples
- $ref: '#/definitions/model'
- $ref: '#/definitions/theme'
- $ref: '#/definitions/connectors'
- $ref: '#/definitions/sources'
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should remove source too? As sidd mentioned lesser pages the better

@royendo royendo changed the title Update the auto YAML Generation [DOCS] Update the auto YAML Generation Aug 21, 2025
@royendo royendo requested a review from NamanMahor August 25, 2025 15:34
@royendo royendo merged commit fe92927 into main Aug 28, 2025
13 checks passed
@royendo royendo deleted the docs/auto-gen-YAML branch August 28, 2025 15:39
grahamplata pushed a commit that referenced this pull request Sep 6, 2025
* some changes for connector YAML

unsur eif schema./yaml is maintained or auto generated.

* remove

* separated YAML schema

* nit

* link fixes

* Update generate_project.go

* add sec warning

* motherduck live connector

* remove local file from connector

* nit

* matching file names

* reordered files and added sources YAML and model SQL

* nit, fix links

* comparing with current YAML changes

* adding annotatons back

* gofmt, golint

* fix

* first pass fix, need to review a few other items

* remove toplevel driver

* single project file

all schemas into one including rillyaml. commented out lines.

reworked generate-project.go:
- moved out logic for sample in copy, uses schema sample instead
- adding in line examples

* Update generate_project.go

* good old cursor wiped my functoni

* missing parameters

* midding parameters and inline examples

* god so many false positives from asking cursor to merge files

* adding links

* fixed oneOf

* split data properties to have api excamples

* gonig through files

* returning original format

next will be adding more examples

* ?

* fixing urls,

* apis fix

* formattnig

* separated rill yaml, adv from rest, #### to ###, added component back

* examples

* https://rilldata.slack.com/archives/C07R1EP2CCV/p1754929153966689

makes more sense, less duplicate info

* fixing examples

confirm yaml, comment out sources
  "/Users/royendo/Desktop/GitHub/rill/runtime/parser/schema/project.schema.yaml": [
        "alerts/**/*.yaml",
        "apis/**/*.yaml",
        "connectors/**/*.yaml",
        "dashboards/**/*.yaml",
        "metrics/**/*.yaml",
        "models/**/*.yaml",
        // "sources/**/*.yaml"
     ]
    }

* overwrite /reference/project-files and fixing links

* type and fix links, add openai connector

* gm fmt and lint

* prettier and lint

* Update docusaurus.config.js

* advanced-model to model

* ready to check

* adding schema_name for MD

* as requested

* fix links

* golang lint

* no more type error

* last touches and requests

* Update apis.md

* lint fix

* remove commented code

* Update Makefile

Co-authored-by: Naman Mahor <[email protected]>

* Update generate_project.go

* lint

* Update generate_project.go

---------

Co-authored-by: Naman Mahor <[email protected]>
kaspersjo pushed a commit that referenced this pull request Sep 11, 2025
* some changes for connector YAML

unsur eif schema./yaml is maintained or auto generated.

* remove

* separated YAML schema

* nit

* link fixes

* Update generate_project.go

* add sec warning

* motherduck live connector

* remove local file from connector

* nit

* matching file names

* reordered files and added sources YAML and model SQL

* nit, fix links

* comparing with current YAML changes

* adding annotatons back

* gofmt, golint

* fix

* first pass fix, need to review a few other items

* remove toplevel driver

* single project file

all schemas into one including rillyaml. commented out lines.

reworked generate-project.go:
- moved out logic for sample in copy, uses schema sample instead
- adding in line examples

* Update generate_project.go

* good old cursor wiped my functoni

* missing parameters

* midding parameters and inline examples

* god so many false positives from asking cursor to merge files

* adding links

* fixed oneOf

* split data properties to have api excamples

* gonig through files

* returning original format

next will be adding more examples

* ?

* fixing urls,

* apis fix

* formattnig

* separated rill yaml, adv from rest, #### to ###, added component back

* examples

* https://rilldata.slack.com/archives/C07R1EP2CCV/p1754929153966689

makes more sense, less duplicate info

* fixing examples

confirm yaml, comment out sources
  "/Users/royendo/Desktop/GitHub/rill/runtime/parser/schema/project.schema.yaml": [
        "alerts/**/*.yaml",
        "apis/**/*.yaml",
        "connectors/**/*.yaml",
        "dashboards/**/*.yaml",
        "metrics/**/*.yaml",
        "models/**/*.yaml",
        // "sources/**/*.yaml"
     ]
    }

* overwrite /reference/project-files and fixing links

* type and fix links, add openai connector

* gm fmt and lint

* prettier and lint

* Update docusaurus.config.js

* advanced-model to model

* ready to check

* adding schema_name for MD

* as requested

* fix links

* golang lint

* no more type error

* last touches and requests

* Update apis.md

* lint fix

* remove commented code

* Update Makefile

Co-authored-by: Naman Mahor <[email protected]>

* Update generate_project.go

* lint

* Update generate_project.go

---------

Co-authored-by: Naman Mahor <[email protected]>
kaspersjo pushed a commit that referenced this pull request Oct 7, 2025
* some changes for connector YAML

unsur eif schema./yaml is maintained or auto generated.

* remove

* separated YAML schema

* nit

* link fixes

* Update generate_project.go

* add sec warning

* motherduck live connector

* remove local file from connector

* nit

* matching file names

* reordered files and added sources YAML and model SQL

* nit, fix links

* comparing with current YAML changes

* adding annotatons back

* gofmt, golint

* fix

* first pass fix, need to review a few other items

* remove toplevel driver

* single project file

all schemas into one including rillyaml. commented out lines.

reworked generate-project.go:
- moved out logic for sample in copy, uses schema sample instead
- adding in line examples

* Update generate_project.go

* good old cursor wiped my functoni

* missing parameters

* midding parameters and inline examples

* god so many false positives from asking cursor to merge files

* adding links

* fixed oneOf

* split data properties to have api excamples

* gonig through files

* returning original format

next will be adding more examples

* ?

* fixing urls,

* apis fix

* formattnig

* separated rill yaml, adv from rest, #### to ###, added component back

* examples

* https://rilldata.slack.com/archives/C07R1EP2CCV/p1754929153966689

makes more sense, less duplicate info

* fixing examples

confirm yaml, comment out sources
  "/Users/royendo/Desktop/GitHub/rill/runtime/parser/schema/project.schema.yaml": [
        "alerts/**/*.yaml",
        "apis/**/*.yaml",
        "connectors/**/*.yaml",
        "dashboards/**/*.yaml",
        "metrics/**/*.yaml",
        "models/**/*.yaml",
        // "sources/**/*.yaml"
     ]
    }

* overwrite /reference/project-files and fixing links

* type and fix links, add openai connector

* gm fmt and lint

* prettier and lint

* Update docusaurus.config.js

* advanced-model to model

* ready to check

* adding schema_name for MD

* as requested

* fix links

* golang lint

* no more type error

* last touches and requests

* Update apis.md

* lint fix

* remove commented code

* Update Makefile

Co-authored-by: Naman Mahor <[email protected]>

* Update generate_project.go

* lint

* Update generate_project.go

---------

Co-authored-by: Naman Mahor <[email protected]>
- name
additionalProperties: false
minItems: 1
explore_properties:
Copy link
Contributor

Choose a reason for hiding this comment

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

This had all the explore properties that can be set for explores defined inline under explore: in a metrics view (the properties here are subtlety different from those in the stand-alone explore resource).

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.

5 participants