Skip to content

Conversation

@Jeehut
Copy link
Contributor

@Jeehut Jeehut commented Jul 26, 2025

Note that this should only be merged AFTER https://github.com/TelemetryDeck/web/pull/2474 has been merged. That PR simplifies the "Version Control" feature (removing it) which is why in the docs I simply wrote that people can duplicate a notebook if they want to share two different versions of an analysis. This should keep things simple and yet flexible.

@Jeehut Jeehut requested review from voidedmain and winsmith July 26, 2025 08:18
Copy link
Contributor

@winsmith winsmith left a comment

Choose a reason for hiding this comment

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

Looks perfect. I have no clue why vale is unhappy with "multiple" and "unpublished", but you can add that and "UUID" to https://github.com/TelemetryDeck/docs/blob/main/.styles/config/vocabularies/Base/accept.txt to silence these spelling warnings if you feel like going the extra mile.

@Jeehut Jeehut merged commit 6c859a3 into main Jul 28, 2025
1 check passed
@Jeehut Jeehut deleted the feature/public-notebooks branch July 28, 2025 13:00
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.

3 participants