Skip to content
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

Docs/SDK/notebooks with nbdev [try 2] #103

Closed
wants to merge 30 commits into from
Closed

Docs/SDK/notebooks with nbdev [try 2] #103

wants to merge 30 commits into from

Conversation

brunosan
Copy link
Member

This take into account some of @weiji14 comments on #102 (which I abandoned to not deal with a polluted git history).

I will not spend much time on the architecture (nbdev vs other options). I do like the option to have all 3 assets at once (documentation htmls, SDK and notebooks) but I understand nbdev might be too rare on stale.

I'll focus meanwhile on the notebooks themselves and we can port the content to other tools.

In the end we need documentation asap.

@brunosan brunosan marked this pull request as draft December 27, 2023 09:49
brunosan and others added 27 commits December 27, 2023 10:58
@brunosan
Copy link
Member Author

After thinking about it, I'll spin out this folder with documentation to a new repository.
I think that our goal to create a good comprehensive documentation, templates, ... will bloat this repo with significantly functionally different content.

Spinning out on https://github.com/Clay-foundation/documentation/pull/1
I'll clean up the publish workflow on a PR next

@brunosan brunosan closed this Dec 28, 2023
brunosan added a commit that referenced this pull request Dec 28, 2023
We are spinning out the documentation to a separate repo.
This cleans up the workflow.

See #103 (comment)
@brunosan brunosan mentioned this pull request Dec 28, 2023
@brunosan brunosan deleted the nbdev2 branch December 28, 2023 15:32
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.

1 participant