This repository was archived by the owner on Mar 2, 2022. It is now read-only.
Replies: 1 comment 1 reply
-
|
At least in Goland, you are able to associate .bubbly files with HCL, which brings along all of the nice default syntax highlighting and code completion. But I like the idea of having a consistent naming convention for sure! +1 for schema instead of tables :) |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Currently, we have a number of ways we name schemas and mock data files in our tests. I propose that we agree on a single style. Because consistency rules, obviously.
I think this is a good question for the Discussion format?
Current situation
For the schema we have:
schema.bubblyinintegration/testdata/schema/tables.hclinstore/testdata/tables[0-5].hclinstore/testdata/sqlgen/tutorial1.schemainwebsite/docs/tutorials/For the data we have:
data.hclinstore/testdata/data[0-5].hclinstore/testdata/sqlgen/Discussion points
1. File name extension
One aspect of this is the file name extension: do we want to use custom
.bubblyor generic.hcl?➕ of
.bubblyis visibility.➖ of
.bubblyis no syntax highlighting by default.➕ of
.hclis the syntax highlighting turned on by default in VSCode and GitHub.Examples:
schema.bubblytables.hclMy vote is for the generic extension
.hclas this would guarantee syntax highlighting in various editors and other tools, both online and offline.2. File name
For schema, we have two obvious choices:
schemaandtables. My vote is forschema, as the Bubbly Schema is one of the core concepts in Bubbly.For data, an obvious choice is
dataand we have little diversity on this now, so it's not really an issue.Proposed action
I am, obviously, open to other ideas!
But, if there is no strong objections, I propose to:
schema.bubblytoschema.hclinintegration/testdata/schema/tables.hcltoschema.hclinstore/testdata/tables[0-5].hcltoschema[0-5].hclinstore/testdata/sqlgen/tutorial1.schematotutorial-1-schema.hclinwebsite/docs/tutorials/And to stick to
schema{.*}.hclfor our other schema adventures 🙃What do you think, @benmarsden @jlarfors @nate-droid ?
Beta Was this translation helpful? Give feedback.
All reactions