-
Notifications
You must be signed in to change notification settings - Fork 2
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
Documentation Standard Updates #23
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made some edits. Looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fivetran-joemarkiewicz One quick question before I approve, one the be checkboxes say Renaming **models to **tables** (only for transform repos)
. Is that meant that all instances of "models" should be "tables" or just in the table with the table and its description? For example, L22 calls them models instead of tables.
@fivetran-catfritz great callout. We will want to make sure the wording rendered in the Quickstart UI is referring to "tables". So in your callout we will want to make that update. But we can leave the other non UI references as models. I just made the update. Let me know if you have other questions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fivetran-joemarkiewicz thanks for the explanation. lgtm!
This pull request was created automatically 🎉:
.github/
files have been properly updated (templates are updated and auto releaser is included).quickstart/quickstart.yml
file aspublic_models
(only for transform repos)**models
to**tables**
(only for transform repos)