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
788 clarify docs intro #797
788 clarify docs intro #797
Conversation
fitting more on the intro page is more important than nice code layout.
fitting more on the intro page is more important than nice code layout.
…into 788-clarify-docs-intro
✅ Deploy Preview for dlt-hub-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@adrianbr . Thank you for helping through this contribution. Here's a draft PR. I appreciate feedback on the content. I won't be offended if you like the original version better. Re the CI/CD process:
|
Yes will do in next few hours
…On Sat, 2 Dec 2023 at 1:32 am, Alena Astrakhantseva < ***@***.***> wrote:
***@***.**** requested changes on this pull request.
------------------------------
In docs/website/docs/intro.md
<#797 (comment)>:
>
-How the script works?: It extracts data from a
-[source](general-usage/glossary.md#source) (here: **chess.com REST API**), inspects its structure to create a
-[schema](general-usage/glossary.md#schema), structures, normalizes and verifies the data, and then
-loads it into a [destination](general-usage/glossary.md#destination) (here: **duckdb** into a database schema **player_data** and table name **player**).
+What that code does:
+
+1. Extracts data from a
+[source](general-usage/glossary.md#source) (here: **chess.com REST API**)
could you add the dots "." at the end of sentences
1. Extracts data from a
[source.]
2. dlt inspects the data's structure to create a
[schema.]
—
Reply to this email directly, view it on GitHub
<#797 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ANX27CRPJ5CWBDBU72QGQILYHHS7PAVCNFSM6AAAAABAATQ4A2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONJZHEZTONZZGI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
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.
update your branch please, i added hot fix to master to skip failed test (it requires credentials)
@AstrakhantsevaAA , I merged the branch. |
Description
Minor rewrite of the intro example
Related Issues
Additional Context
1st PR by new contributor, so some of the motivation was simply to contribute something.
It could be good to have this example (chess) as a Google Colab too. I was nicely suprised that Google Colab just works, with only a simple google account. But the static html example on intro.md is still great for people casually browsing the docs on their phone on the bus etc.